-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Add aria attributes to dummy input #3090
Conversation
@JedWatson is it possible to get this added into the library? Not having the ability to add aria labels to non-searchable inputs makes this library unsuitable for projects required to meet modern accessibility standards. |
Can we revisit this fix? It would be great to get this in to solve accessibility issues with this component. |
🦋 Changeset is good to goLatest commit: 910b944 We got this. Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Just wondering if this update will make it into the next release. Also wondering when next release would be? |
When
props.isSearchable
isfalse
, aria attributes likearia-label
andaria-labelledby
are not passed through to theDummyInput
control that takes place of the input text box. Since the dummy component can't be replaced, making a non-searchable select breaks accessibility even though there are standards patterns for non-editable combo boxes.Since
react-select
consumers have control over both the accessibility props andisSearchable
without control overDummyInput
, they should remove the accessibility props from the dummy input themselves by changing<Select>
props.