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

feat: add keyboard navigation for resource tree #2591

Merged

Conversation

mstykow
Copy link
Collaborator

@mstykow mstykow commented Mar 5, 2024

Summary of changes

  • keep track of focused ID instead of focused index in Virtuoso
  • add focus state for tree nodes and define keyboard actions

Context and reason for change

closes #2572

How can the changes be tested

Navigate the resource browser up and down and left and right and check that focus behaves as expected.

@mstykow mstykow linked an issue Mar 5, 2024 that may be closed by this pull request
@mstykow mstykow force-pushed the feat-list-keyboard-navigation branch from 109cd84 to b95ffda Compare March 5, 2024 19:35
@mstykow mstykow force-pushed the feat-tree-keyboard-navigation branch from 3be3c17 to c65d5e0 Compare March 5, 2024 19:36
@mstykow mstykow force-pushed the feat-list-keyboard-navigation branch from b95ffda to 18d42f1 Compare March 6, 2024 10:08
@mstykow mstykow force-pushed the feat-tree-keyboard-navigation branch from c65d5e0 to 6deb431 Compare March 6, 2024 10:08
@MarkusObendrauf MarkusObendrauf force-pushed the feat-tree-keyboard-navigation branch 3 times, most recently from a8ded4b to 05aa69a Compare March 6, 2024 12:44
@MarkusObendrauf MarkusObendrauf marked this pull request as ready for review March 6, 2024 14:37
@mstykow mstykow force-pushed the feat-list-keyboard-navigation branch 2 times, most recently from 29cd131 to 50a0462 Compare March 6, 2024 17:05
- keep track of focused ID instead of focused index in Virtuoso
- add focus state for tree nodes and define keyboard actions

closes #2572

Signed-off-by: Maxim Stykow <maxim.stykow@tngtech.com>
Signed-off-by: Markus Obendrauf <markus.obendrauf@tngtech.com>
@mstykow mstykow force-pushed the feat-tree-keyboard-navigation branch from 05aa69a to 91bf5fa Compare March 6, 2024 17:06
Copy link
Collaborator Author

@mstykow mstykow left a comment

Choose a reason for hiding this comment

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

Co-authored this PR with @MarkusObendrauf and I approve.

@mstykow mstykow merged commit 23173fd into feat-list-keyboard-navigation Mar 6, 2024
5 checks passed
@mstykow mstykow deleted the feat-tree-keyboard-navigation branch March 6, 2024 17:20
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.

Enable keyboard navigation in resource browser
1 participant