-
-
Notifications
You must be signed in to change notification settings - Fork 299
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
Additional keyboard navigation scenario #651
Labels
Example
A good example of a certain scenario
Comments
That's a good example, thank you very much for sharing it. I am not sure if I wish I had more time to spend on the details of accessible lists :(. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I was having a play around with your keyboard navigation, and I want to suggest an improvement.
Problem: When a user uses the keyboard to navigate the list, the list item should be in focus.
Solution: Create a roving focus prop using the index and the current item index in a separate inner item.
Here is the code based on the original keyboard navigation to do it: https://codesandbox.io/s/react-virtuoso-focused-item-jv35gp?file=/App.js
I'm happy to make a PR if this is of interest.
The text was updated successfully, but these errors were encountered: