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
ENH: Use bootstrap tooltip to display icons title #924
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM just one comment about the tooltip text for the mode switcher.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, we're already loading the bootstrap JS code so it makes sense to utilize this feature IMO.
Fix #920
From a design point of view that's a bad practice to use icons without title as it leave the user interprete what the icon is for. Even if we use stadards of the web it's better to be on the safe side.
Titles where already set for the generated icons but missing for the search and theme switcher.
I also changed their design simply by activating the bootstrap tooltip instead if relying on browser display.