-
Notifications
You must be signed in to change notification settings - Fork 355
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: investigate how to improve screen reader experience and update #5383
Labels
Comments
be sure our examples meet a11y standards. i.e. they dont put tooltips on static elements |
tooltips which change dynamically need to be accessible: #1655 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the issue. What is the expected and unexpected behavior?
I've noticed that our tooltip examples aren't always announced in VO. The only example that has solidly worked when I've tested lately has been the Options example, and my theory behind this is the placement in the DOM. We should investigate how we can make sure that our tooltips work in a screen reader (I'd like us to ideally also test this in other screen readers as well like NVDA and JAWS). I'm wondering if we can treat it similarly to how we treat alerts and add an
aria-live
to let screen readers know where the content will be updated and make sure it's aware of these changes.Please provide the steps to reproduce. Feel free to link CodeSandbox or another tool.
Open VO and test our tooltip examples. In Chrome, the first example works along with the Options example. In Safari, only the Options example works. I'd like to make sure all implementations of tooltip work so that we can ensure our products' tooltips work in their implementations.
Is this a bug or enhancement? If this issue is a bug, is this issue blocking you or is there a work-around?
bug
Remaining issues in this epic:
disabled
attribute from the controls that have it, so that instead they're only aria-disabled. The tooltip does de-render once focus is moved out of this controls group, though.No longer applicable:
Completed issues:
The text was updated successfully, but these errors were encountered: