Skip to content
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(1030): adds a customizable property for aria-label #1765

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ma31cm
Copy link

@ma31cm ma31cm commented Apr 3, 2023

The aria-label for the search input is currently hard-coded to "Search for option". This causes problems for screen reader users and fails WCAG 2.1 criteria 2.5.3 and 3.3.1.

This PR adds a new property 'ariaLabel' that can be used to change the value of aria-label for the search input. This fixes (or at least partially fixes) the following issues:

@merziy
Copy link

merziy commented Jun 9, 2023

Would love to see this implemented! Running into a related issue trying to get a site that uses vue-select with ADA compliance.

@howard-tzw
Copy link

@ma31cm @merziy Hello, if you are using Vue 3, welcome to submit a pull request to the following forked repository.

https://github.com/howard-tzw/vue3-select

I have available time to maintain this project, but it may be limited to the Vue 3 version.

@merziy
Copy link

merziy commented Jun 12, 2023

@howard-tzw Thank you. @ma31cm Is this something you have time to do?

@howard-tzw
Copy link

howard-tzw commented Jun 21, 2023

@merziy You can now use the property of this PR in vue3-select@0.0.8, thanks to @ma31cm

@ma31cm
Copy link
Author

ma31cm commented Aug 10, 2023

@howard-tzw Sorry for the slow reply - somehow your comment completely passed me by! But thank you, that's great 😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants