-
Notifications
You must be signed in to change notification settings - Fork 27
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
accessibility, focus event listener #11
Comments
I'd love to add support for this! Maybe we could make it opt-in at first? |
Yeah, that was my inclination as well! I'll think about it a bit more and open a PR when I get a chance. |
I think is a great idea. One way tabbing could be handled is some kind of timer -- if a user navigating via tab pauses on an element for a given period of time, the element loads. I foresee a couple of issues with this:
I'm definitely going to think about this more also, but I think that adding a focus listener to this would not only improve its accessibility but also encourage the developers who use this to be more intentional about how they're setting up their |
What do you think about adding focus event listener for non-mouse users? I'm undecided myself if this is a good idea but I was thinking of ways to make this accessible. The problem with a focus listener is that a user tabbing through items on the page would trigger everything as they navigate with the keyboard.
Thoughts? Concerns?
Great idea by the way!
The text was updated successfully, but these errors were encountered: