-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
Improve accessibility for keyboard users #106
Labels
Comments
@amclark42 Thank you very much for your clear report! We made some last minute fixes to logging in flow, and I agree that it left some warts where improvements could be made. Your report leaves a clear roadmap for where they are needed! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
My setup:
I prefer to use keyboard navigation because it's easier on my wrists. I've struggled with using the new Dashboard in this way: I can't always see where the focus is on the page, and some important controls can't be tabbed to at all.
As an example, starting from exist/apps/dashboard/index.html:
<button>
s. Since it looks like a button, it might be better to use<button>
, or make sure that default button functionality is available on the<a>
. But it's available for interaction, which is what really matters.)While most of this is the same when using Chrome, I can tab back to the User field there, strangely enough.
I hope this breakdown helps. I like the new Dashboard a lot; it feels much cleaner and nuanced in what it displays and when. The old Dashboard still wins in terms of keyboard accessibility, unfortunately—maybe because it uses standard HTML elements, so that it can rely on the browsers' default behaviors? There's a lot more you have to build into custom elements, I think.
The text was updated successfully, but these errors were encountered: