Skip to content
This repository was archived by the owner on Oct 2, 2019. It is now read-only.

Added aria-expanded to the multiple select template #1776

Merged

Conversation

david-kirkland
Copy link
Contributor

Closes #1775

@user378230
Copy link
Contributor

What about the select2 and selectize templates? Are they already correct?

@david-kirkland
Copy link
Contributor Author

david-kirkland commented Aug 19, 2016

There are lots of ADA issues with UI Select. With that said, I ran aXe (plugin mentioned in #1775) on the Plunker demos for Select 2 and Selectize. It did not detect this particular issue (aria-expanded needs to be on elements with role="combobox"), so that's it for this PR.

@user378230 user378230 merged commit 6766754 into angular-ui:master Aug 22, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants