Skip to content

make sample table sequentially focusable when no cell is active#1101

Merged
GaelVaroquaux merged 4 commits intoskrub-data:mainfrom
jeromedockes:tab-sample-table
Oct 7, 2024
Merged

make sample table sequentially focusable when no cell is active#1101
GaelVaroquaux merged 4 commits intoskrub-data:mainfrom
jeromedockes:tab-sample-table

Conversation

@jeromedockes
Copy link
Copy Markdown
Member

currently before clicking on any of the cells it is not possible to bring focus to the sample table with the keyboard. this PR makes the table sequentially focusable (with Tab); when it receives focus it activates the first cell

@jeromedockes jeromedockes added this to the 0.4.0 milestone Oct 7, 2024
@jeromedockes jeromedockes marked this pull request as ready for review October 7, 2024 08:09
Copy link
Copy Markdown
Member

@glemaitre glemaitre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice. If you can only solve the merge conflict.

@GaelVaroquaux
Copy link
Copy Markdown
Member

I fixed the merge conflict. Launching auto-merge

@GaelVaroquaux GaelVaroquaux enabled auto-merge (squash) October 7, 2024 09:12
@GaelVaroquaux GaelVaroquaux merged commit 1aec315 into skrub-data:main Oct 7, 2024
@jeromedockes jeromedockes deleted the tab-sample-table branch October 7, 2024 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants