-
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(ComboBox): removes all DAP violations #4201
fix(ComboBox): removes all DAP violations #4201
Conversation
Deploy preview for carbon-elements failed. Built with commit 12d75be https://app.netlify.com/sites/carbon-elements/deploys/5d9d1c5cbfd7450008af7d3c |
Deploy preview for the-carbon-components ready! Built with commit 12d75be https://deploy-preview-4201--the-carbon-components.netlify.com |
Deploy preview for carbon-components-react ready! Built with commit 12d75be https://deploy-preview-4201--carbon-components-react.netlify.com |
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.
🎉
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 👍 - Thanks @abbeyhrt!
(Finally) Closes #3260
Adds isOpen check in
aria-owns
andaria-controls
oninput
so that it is null if the dropdown is closed. Also changes the color of the numbers in the "items as components' story so it doesn't fail a color contrast check.Changelog
Changed
Testing / Reviewing
Run DAP on the open and closed Combobox and confirm that there are no violations.