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

[Accessibility] Add aria-controls #1798

Open
damian-krupa opened this issue Sep 24, 2023 · 2 comments
Open

[Accessibility] Add aria-controls #1798

damian-krupa opened this issue Sep 24, 2023 · 2 comments

Comments

@damian-krupa
Copy link

I am using an automated accessibility testing tool called Level Access Assistant. It shows an issue for Select.vue component - aria-controls attribute is lacking.

Describe the solution you'd like
add this line of code :aria-controls="vs${uid}__listbox" in the Select.vue combobox div, right under aria-owns

Additional context
Error message: This DIV (role=combobox) must have all of the following attributes: aria-controls | aria-expanded

@damian-krupa
Copy link
Author

I have opened a PR some time ago to fix this. Please take a look, it's a one-liner #1802

@legostud
Copy link

legostud commented Jun 3, 2024

That looks like the same bug that I'm running into with my audit.

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

No branches or pull requests

2 participants