-
Notifications
You must be signed in to change notification settings - Fork 229
docs(tabs-overflow): update a11y docs #5621
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
Conversation
|
📚 Branch Preview🔍 Visual Regression Test ResultsWhen a visual regression test fails (or has previously failed while working on this branch), its results can be found in the following URLs:
Deployed to Azure Blob Storage: If the changes are expected, update the |
33438a3
to
9106180
Compare
Tachometer resultsCurrently, no packages are changed by this PR... |
e75d965
to
48eb830
Compare
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.
Would we want to mention something about how tabs overflow sets aria labels on the action buttons? It looks like the action buttons get labeled with "scroll to previous tabs" and "scroll to next tabs", that feels like it could be a nice accessibility feature to mention here.
Great callout! I went ahead and added a note on that. ✨ |
e70e892
to
311aeda
Compare
Description
Improving the accessibility documentation of components.
Related issue(s)
SWC-415
Author's checklist
Reviewer's checklist
patch
,minor
, ormajor
featuresDevice review