-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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(select): a11y: stop SR announcement for hidden labels #18061
Conversation
✅ Deploy Preview for v11-carbon-web-components ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for carbon-elements ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for v11-carbon-react ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #18061 +/- ##
=======================================
Coverage 82.19% 82.19%
=======================================
Files 404 404
Lines 14151 14151
Branches 4468 4468
=======================================
Hits 11631 11631
Misses 2359 2359
Partials 161 161 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
Gower confirmed in CAG call that the label has to be announced, otherwise that might lead to a11y issues. |
Closes #16613
SR should not announce the label when
hideLabel
is trueChangelog
New
Added
aria-hidden
attributeTesting / Reviewing
Open From the Select - playground story
set the hide label Boolean to "True" and observe the SR do not announce label.