Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

Material-sidenav tab order and focus needs to be handled #330

Closed
marcysutton opened this issue Sep 26, 2014 · 3 comments
Closed

Material-sidenav tab order and focus needs to be handled #330

marcysutton opened this issue Sep 26, 2014 · 3 comments
Labels
a11y This issue is related to accessibility

Comments

@marcysutton
Copy link
Contributor

To be accessible, material-sidenav needs to hook into transition events to hide and show the component with display: none after animating. This will remove interactive controls from the tab order when they are not in use, which is easier and more performant than changing the tabIndex of every interactive child element.

In addition, focus should be sent into the sidenav when it opens, directing the user's attention to new content on screen. This helps a desktop keyboard user with a smaller screen, as well as a mobile screen reader user. The pattern is very similar to material-dialog.

@marcysutton marcysutton added the a11y This issue is related to accessibility label Sep 26, 2014
@ajoslin
Copy link
Contributor

ajoslin commented Oct 17, 2014

Added display: none via 105b0e0.

@ajoslin
Copy link
Contributor

ajoslin commented Oct 17, 2014

@ajoslin
Copy link
Contributor

ajoslin commented Oct 17, 2014

Done!

@ajoslin ajoslin closed this as completed Oct 17, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a11y This issue is related to accessibility
Projects
None yet
Development

No branches or pull requests

2 participants