-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
fix(module:Tooltip) Tabindex #2404
fix(module:Tooltip) Tabindex #2404
Conversation
Applies a tabindex to the wrapping div to help with focus driven from tabbing.
Codecov Report
@@ Coverage Diff @@
## master #2404 +/- ##
==========================================
+ Coverage 27.81% 29.75% +1.94%
==========================================
Files 496 519 +23
Lines 33146 25403 -7743
Branches 0 247 +247
==========================================
- Hits 9218 7559 -1659
+ Misses 23928 17804 -6124
- Partials 0 40 +40
Continue to review full report at Codecov.
|
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.
Fantastic!
…ip with tab key (#2404) Applies a tabindex to the wrapping div to help with focus driven from tabbing.
…ip with tab key (#2404) Applies a tabindex to the wrapping div to help with focus driven from tabbing.
…ip with tab key (#2404) Applies a tabindex to the wrapping div to help with focus driven from tabbing.
Applies a tabindex to the wrapping div to help with focus driven from tabbing.
🤔 This is a ...
🔗 Related issue link
💡 Background and solution
The Tooltip does not allow keyboard-based users to access their content because it's not a tab-able element. Adding a
tabindex
to thediv
that's handling the events fixes this.div
that handles the event listeners.📝 Changelog
Users will be able to tab to elements that leverage the Tooltip component.
☑️ Self Check before Merge