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

Tooltip: moving hover from tooltip back to trigger closes tooltip #10240

Closed
jlukosky opened this issue Aug 22, 2019 · 9 comments
Closed

Tooltip: moving hover from tooltip back to trigger closes tooltip #10240

jlukosky opened this issue Aug 22, 2019 · 9 comments

Comments

@jlukosky
Copy link

Environment Information

  • Package version(s): (fill this out)
  • Browser and OS versions: (fill this out)

Note that our screen reader support is limited to Edge + Narrator. Issues that do not reproduce in Edge + Narrator, and aren't caused by obvious invalid aria values, should be filed with the respective screen reading software, not the Fabric repo.

Describe the issue:

When moving hover from tooltip back to trigger the tooltip closes. The trigger shouldn't close the tooltip, it's behavior should be consistent. As it is the trigger is serving as a show/hide toggle.

Please provide a reproduction of the issue in a codepen:

Actual behavior:

Moving hover from tooltip to trigger closes tooltip.

Expected behavior:

Moving hover from tooltip to trigger should not close the tooltip

Documentation describing expected behavior

The expectation is that the behavior will be consistent: hovering the trigger should show the tooltip. As it is now sometimes hovering the trigger shows the tooltip, sometimes it hides it.

@maxwellred
Copy link
Contributor

Hi @jlukosky . Thanks for opening this issue about Tooltip and congratulations on this being your first in UI Fabric React! I have contacted the code owner in regards to this behavior and will be attending to it shortly.

@micahgodbolt
Copy link
Member

Having tooltips be sticky when hovered isn't a 100% normal HTML experience, so we rely on our designers to inform us on how it should work. The fix shouldn't be too hard to do, but I want to first confirm that this is the correct design choice before doing the work.

@betrue-final-final do you have any context on if tooltip should stay open after hovereing over the tooltip and then back onto the button? @jspurlin do you know what the expectation is on the ribbon with tooltips?

@jspurlin
Copy link
Collaborator

I know in the desktop apps, the user can hover over the tooltip and then back over the button (but windows and native tooltips in the browser don't do that). This sounds like a feature request that should be vetted with designers.

@jspurlin
Copy link
Collaborator

@micahgodbolt actually, looking at WCAG 2.1 there is now a requreiment that the user can hover over tooltips without them disappearing: https://www.w3.org/TR/WCAG21/#content-on-hover-or-focus Specifically:

Hoverable
If pointer hover can trigger the additional content, then the pointer can be moved over the additional content without the additional content disappearing;

@micahgodbolt
Copy link
Member

@jspurlin yeah, we now support tooltips staying when you hover over them. Question is what happens after focus is placed back on button. should tooltip stay? should it dissapear? Should it dissapear and then come back after regular delay?

@jlukosky
Copy link
Author

@jspurlin yeah, we now support tooltips staying when you hover over them. Question is what happens after focus is placed back on button. should tooltip stay? should it dissapear? Should it dissapear and then come back after regular delay?

I think the issue here is on hover not on focus. Placing focus back on the button would not be at issue in this case. If the user starts using the keyboard after hovering to reveal the tooltip of course the tooltip would be hidden until the focus is moved on the trigger.

Focus can't directly move from the tooltip to the trigger since tooltips don't support focus.

@kenotron
Copy link
Member

kenotron commented Sep 3, 2019

This needs spec'ing before proceeding, I'm removing myself and Max off of the assignee in this case leaving the backlog review label.

@msft-fluent-ui-bot
Copy link
Collaborator

Because this issue has not had activity for over 150 days, we're automatically closing it for house-keeping purposes.

Still require assistance? Please, create a new issue with up-to date details.

@msft-fluent-ui-bot msft-fluent-ui-bot added the Resolution: Soft Close Soft closing inactive issues over a certain period label Apr 26, 2021
@jlukosky
Copy link
Author

This should be reopened and resolved. If this isn't addressed it is a violation of 1.4.13: Content on Hover or Focus

@microsoft microsoft locked as resolved and limited conversation to collaborators May 27, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

7 participants