-
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: add area-describedby to components that use helperText #13371
fix: add area-describedby to components that use helperText #13371
Conversation
✅ Deploy Preview for carbon-components-react ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for carbon-elements ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
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.
Tested all components and all are working as intended.
One little thing, do we want to add helperText to the Playground Argtypes for DatePicker?
helperText: {
control: { type: 'text' },
table: {
category: 'DatePickerInput',
},
},
done! |
…bedby-to-link-help-text-with-component
@francinelucca LGTM! one thing I just merged in |
…722-a11y-dropdown-does-not-use-aria-describedby-to-link-help-text-with-component
…bedby-to-link-help-text-with-component
…nk-help-text-with-component' of github.com:francinelucca/carbon into 12722-a11y-dropdown-does-not-use-aria-describedby-to-link-help-text-with-component
…bedby-to-link-help-text-with-component
…722-a11y-dropdown-does-not-use-aria-describedby-to-link-help-text-with-component
…nk-help-text-with-component' of github.com:francinelucca/carbon into 12722-a11y-dropdown-does-not-use-aria-describedby-to-link-help-text-with-component
…722-a11y-dropdown-does-not-use-aria-describedby-to-link-help-text-with-component
Resubmitting for review. For RadioButtonGroup and CheckboxGroup I tried setting the so I decided to leave it in the fieldset. I found this proof case that saw voiceover skipping over the aria-describedby in fieldset in VoiceOver while other a11y tools did read it https://russmaxdesign.github.io/accessible-forms/fieldset-error02.html |
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.
LGTM!
…bedby-to-link-help-text-with-component
Closes #12722
Adds
aria-describedby
attribute to components that use helperText:Changelog
Changed
aria-describedby
attribute inMultiselect
,Dropdown
,Combobox
,DatePickerInput
,NumberInput
,Select
,ProgressBar
,TextArea
,ControlledPasswordInput
,PasswordInput
,RadioButtonGroup
Testing / Reviewing
Components should behave and look exactly the same. Voiceover now reads helper text when focused on input