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

Toggle: Improve screen-reader accessibility #1557

Merged
merged 5 commits into from
Apr 20, 2017
Merged

Toggle: Improve screen-reader accessibility #1557

merged 5 commits into from
Apr 20, 2017

Conversation

c-w
Copy link
Contributor

@c-w c-w commented Apr 19, 2017

Pull request checklist

  • Addresses an existing issue
  • Include a change request file if publishing
  • New feature, bugfix, or enhancement
    • Includes tests
  • Documentation update

Description of changes

The Toggle is currently not very accessible for screen-reader users as it doesn't have hooks that let us provide context on the option that's being toggled. This pull request adds two optional aria-labels that lets us provide more context on the on/off states for screen-reader users.

Video of screen-reader behavior after applying the pull-request.

It's not ideal that the screen-reader re-announces the toggle onText and offText, however, this is not easily preventable as many browser/screenreader combinations will announce aria-hidden children while reading the parent.

Focus areas to test

  1. Open Narrator on Windows 10.
  2. Open the Toggle example in Edge.
  3. Navigate to "enabled and checked" toggle.
  4. Hear the screen-reader speak the onAriaLabel.
  5. Press enter to state-switch the toggle.
  6. Hear the screen-reader speak the offAriaLabel.

@c-w
Copy link
Contributor Author

c-w commented Apr 19, 2017

@mikewheaton: Thanks for the quick sign off. I don't have merge-access, so could you please also hit merge before master advances again?

@micahgodbolt micahgodbolt merged commit bf60dbc into microsoft:master Apr 20, 2017
@c-w c-w deleted the bugfix/toggle-screenreader-improvements branch April 20, 2017 16:32
@microsoft microsoft locked as resolved and limited conversation to collaborators Aug 31, 2019
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.

4 participants