-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Allow navigating down from header cells, as well as support cmd/ctrl + arrow keys #10777
Conversation
🪼 branch checks and previews
|
🦄 change detectedThis Pull Request includes changes to the following packages.
With the following changelog entry.
Maintainers or the PR author can modify the PR title to modify this entry.
|
Header issue fixed!
Just to confirm, the new cmd/ctrl + arrow functionality is specifically jump navigation right? Works great. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks @abidlabs!
I'm gonna go ahead and merge this in, hope that's okay! |
Yes great thanks @hannahblair! |
There was a small issue: if you used the keyboard to navigate to the header cell, you couldn't navigate down to the regular table cells.
Before:
Screen.Recording.2025-03-10.at.1.51.53.PM.mov
Fixed:
Screen.Recording.2025-03-10.at.1.52.01.PM.mov
I also added support for navigating with cmd/ctrl+arrow keys, you can navigate through the dataframe pretty easily now:
Screen.Recording.2025-03-10.at.5.35.24.PM.mov
Closes: #7610 (the issue mentions a few bugs, which I've confirmed resolved. The issue also mentions home/end keys, which I'm not including in this PR, as they do not add any new functionality that cmd/ctrl+arrow keys already provide).