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

Keyboard Only Tab Issue - FF Only, Long Lists Only #2469

Closed
wcndave opened this issue Nov 5, 2015 · 6 comments
Closed

Keyboard Only Tab Issue - FF Only, Long Lists Only #2469

wcndave opened this issue Nov 5, 2015 · 6 comments
Labels

Comments

@wcndave
Copy link

wcndave commented Nov 5, 2015

http://jsfiddle.net/uxh723ch/

If you use chosen on a multiple select field, which has more than 10 options (and causes the list to have a scrollbar), Firefox requires two tabs to move to the next element.

On the first tab, there is no on screen focus, and therefore keyboard only users are in what's called a "keyboard trap". They don't know where they are.

This only happens on FF, and it only happens when the options are long enough to create a scrolling list.

See the fiddle link for an example.

Now, I can get this to work if I add tabindex to all elements, however consider a large dynamic web site, with lots of menu and navigation links, and things like "sign in" and "New message", it's just not practical to apply tab index to every possible link / element in the page.

The natural DOM layout should support correct tab order.

Hope someone can help, as I require a high level of accessibility.

Thanks!

@tjschuck
Copy link
Member

tjschuck commented Nov 5, 2015

What version of Firefox? And what OS and OS version are you using?

Can you confirm that the Chosen demo page also shows the issue?

Thanks!

@wcndave
Copy link
Author

wcndave commented Nov 5, 2015

Using version 41.0.2 on windows 7 64 bit.

It does not occur on the chosen demo, because there is a "hard coded" tabindex value on each field.

If I copy your page, remove all tab index attributes, then the issue does indeed occur.

Let me know if you need anything else, we have found a few other accessibility issues when a professional third party reviewed it, and I note some accessibility comments / questions going back a few years now, would it be better to try and deal with them all in one go?

As a power user who uses keyboard mostly, I do find that accessibility is just a good thing for all users. mostly.

Thanks for quick response. Dave

@tjschuck
Copy link
Member

tjschuck commented Nov 5, 2015

Thanks for the follow-up!

Let me know if you need anything else, we have found a few other accessibility issues when a professional third party reviewed it, and I note some accessibility comments / questions going back a few years now, would it be better to try and deal with them all in one go?

Smaller, more focused PRs are definitely more likely to get reviewed and merged, for better or worse.

@wcndave
Copy link
Author

wcndave commented Nov 18, 2015

Any thoughts on this at all? I'd like to know if I need to develop a jQuery process to programatically add tabindex to all items on a page.

Thanks, Dave

@mgifford
Copy link

I just tested this in FF 84.0.1 (64-bit) and didn't find any issues. I suspect that this issue can be closed as it's pretty old and all the code has changed a lot.

@tjschuck
Copy link
Member

@mgifford Thanks for following up! I'm all for stale issue cleanup. We'll reopen if necessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants