-
Notifications
You must be signed in to change notification settings - Fork 490
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
Display help when pressing '?' #756
Comments
What help are you thinking of here? A list of keyboard commands? |
@olizilla Yes, supported keyboard shortcuts. |
I wonder if we could support moving to menu navigation via left/right arrows. cc @terichadbourne https://github.com/protocol/design/issues/145 |
I love the focus on keyboard users here. In general, I presume that sticking to the common methods they use (for example, tab to focus on the next thing on the site and enter to select it) will be better than creating new site-specific shortcuts, but I'm not clear on what part of the site interaction you're referring to, so feel free to ignore this comment if it's irrelevant. |
To illustrate:
|
Do we have any accessibility gurus? I'm not one, but i know that tabbing is core to the way keyboard-based users get around, so that's also a consideration. |
My accessibility training is minimal, but what I would have expected on a page like this with left-side nav is that when you tabbed from the top of the page you would first cycle through all of the nav options (where then clicking or hitting enter would change the content on the right) and then cycle through all of the clickable content on the right side of the page (main content). I don't think it's necessary to build a special command to tab from left to right. Just think of nav as top in the cycling, and make sure it's in nav tags so people with various accessibility tools can skip it or access it in the ways they're used to. |
Closing in favor of accessibility master issue ipfs/ipfs-gui#28 -- this discussion has been linked to from there so we avoid it getting lost to history. |
Current shortcuts:
/cc @lidel
The text was updated successfully, but these errors were encountered: