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

Add tooltips to buttons with dynamic visibility support #825

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

tanzoniteblack
Copy link

I was taking a look at #405 and thought that a decent middle ground on solving this is instead of adding a new button to hold a variable that will dictate whether to show tooltips, we could instead use hovering while holding a modifier key on the keyboard to control that display logic.

This PR adds integrated tooltips across score control buttons to provide contextual guidance to users, but only when the Control key is being held & the user is actively hovering over an element with the class tooltip that has a child element with the class tooltiptext. I couldn't add the tooltiptext span directly as a child of the button or the button's display rules interfere with getting the tooltip to show up on top of the other elements on the page.

Notes:

Because of the way browsers detect key presses, the user will have had to interact with the page in some format before the control + hover functionality will work. This could be pressing a button or even just clicking anywhere on the page.

I am in zero way tied to where this javascript code should live nor what the styling should be, so please feel free to adjust that or make suggestions on what changes I should make!

And finally, I only added a few tooltip texts because I'm still learning how to use this program and am the prime audience for needing these help bubbles!

image

Integrated tooltips across score control buttons to provide contextual guidance to users. Enabled dynamic tooltip visibility when the Control key is pressed, improving accessibility and user experience.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant