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

Further accessibility enhancements #53

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

tylermos
Copy link

Further accessibility improvements to the tagger widget, notably

  1. Adding ariaDescribedBy parameter
  2. Added tag guidance to tell screen reader users how to remove a tag
  3. Added audio guidance to outline the current selection when the widget is first focused
  4. Made selected tags a list to make relationship clear to screen readers
  5. Expose hierarchical levels directly to screen readers

And some basic code clean up that IntelliJ moaned about

matteason and others added 8 commits May 1, 2019 14:05
- Announce when a tag is selected or removed
- Announce number of options available when filtering
- Announce values of selected tags when widget is focused
- Allow tag removal with delete key
- Move aria-expanded from input element to widget div so expanded/collapsed state is announced
Fixed an issue in single-select mode where selecting an item while another is already selected raised an exception. Fixed another issue when adding a freetext item raised an exception
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.

2 participants