fix(a11y): use aria-errormessage for invalid text inputs#21297
Conversation
✅ Deploy Preview for v11-carbon-react ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for v11-carbon-web-components ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for carbon-elements ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #21297 +/- ##
==========================================
- Coverage 95.08% 95.06% -0.02%
==========================================
Files 541 541
Lines 45217 45217
Branches 6560 6560
==========================================
- Hits 42996 42987 -9
- Misses 2092 2101 +9
Partials 129 129
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
adamalston
left a comment
There was a problem hiding this comment.
Are there supposed to be test changes to accompany this fix? I'm wondering what the plan is for preventing regressions.
Co-authored-by: Sangeetha Babu <sangeetha9223@gmail.com>
21677a1
Closes #21247
Changelog
Changed
aria-errormessagefor error messages instead ofaria-describedbyTesting / Reviewing
aria-errormessageattribute pointing to error message IDaria-describedbywhen input is not invalidtoHaveAccessibleErrorMessage()assertion in PlaywrightPR Checklist
As the author of this PR, before marking ready for review, confirm you:
More details can be found in the pull request guide