Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IDX Logged In / Logged Out CSS & Class Names #15

Open
bhubbard opened this issue Nov 9, 2015 · 4 comments
Open

IDX Logged In / Logged Out CSS & Class Names #15

bhubbard opened this issue Nov 9, 2015 · 4 comments

Comments

@bhubbard
Copy link
Contributor

bhubbard commented Nov 9, 2015

Right now, we can style IDX Broker pages using the class names for logged out users. However at the moment there is no way to do this on the WordPress side as its technically a different site. We should add logic to the WordPress plugin to detect if the user was logged into IDX Broker (probably via a cookie), and if so update the body classes with the appropriate new class names:

idxbroker-logged-in

or

idxbroker-logged-out
sheparddw added a commit that referenced this issue Nov 24, 2015
…master

* commit '4c79e08949c21561d91ba395f88f6a66f71a18a4':
  Update readme file.
  Fix bug where the omnibar does not work properly on strict servers.
  Fix an issue with duplicate dynamic wrapper tags when equity is intalled. Fix a bug with errors being reported.
@amcn84
Copy link

amcn84 commented Dec 29, 2015

This seems like it would be better handled by adding this to the widget scripts rather than rely on WordPress as the script would adjust the classes on load based on the particular visitors logged-in/out state.

@bhubbard
Copy link
Contributor Author

I see your point, but ideally I would like to be able to style a page based on IDX login status, even if no widget exists on the page.

For example, a user could browse to a basic about page for the site which has no idx broker widget or features, and is not an IDX Broker page. However in the menu at the top there may be a login button. I should be able to hide this button if we know the user has already logged into IDX Broker.

@AlphaBlossom
Copy link

Any more thought given to this? I'd like to also see this, adding an "IDX-loggedIn" class to the body.

@chrisdavery-arch
Copy link

Hello, is this issue solved? Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants