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

Move aria-multiselectable to [role=listbox] in the Combobox component #2271

Merged
merged 2 commits into from
Feb 10, 2023

Conversation

RobinMalfait
Copy link
Member

This PR fixes a misplacement of the aria-multiselectable aria attribute.

I couldn't find evidence why we originally put it on the [role="combobox"] element, but I think this was because we started from the Listbox component as a starting point.

I also couldn't find evidence why we should put it on the [role="combobox"] itself. Looking at the best practices they mention the aria-multiselectable attribute, and that it should be used in combination with some roles, such as the listbox role. https://w3c.github.io/aria/#aria-multiselectable

Testing it in the browser with VoiceOver: This does indeed improve the information that you get from VO. Luckily all other functionality is exactly the same.

Fixes: #2270

@vercel
Copy link

vercel bot commented Feb 10, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
headlessui-react ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Feb 10, 2023 at 2:41PM (UTC)
headlessui-vue ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Feb 10, 2023 at 2:41PM (UTC)

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