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

Using duplicate shortcut scrolls to the bottom of the tree #52

Open
poirierlouis opened this issue Nov 2, 2024 · 0 comments
Open

Using duplicate shortcut scrolls to the bottom of the tree #52

poirierlouis opened this issue Nov 2, 2024 · 0 comments
Assignees
Labels
bug Something isn't working ux/ui

Comments

@poirierlouis
Copy link
Owner

poirierlouis commented Nov 2, 2024

Problem:
Using CTRL + D / CMD + D on a selected widget will force scrolling at the bottom of the tree. This is not expected, the scroll offset should stay the same.

Lead:
Default behavior of key D is to scroll down at the bottom. Using event.preventDefault() should be enough to fix this issue.

Scrolling when key D is pressed is related to a11y of <mat-tree>. It will scroll to the next node starting with the letter D. This can be verified with other keys/letters.

First migrate to latest version of <mat-tree>, using levelAccessor/childrenAccessor instead of treeControl. As of 18.2.11, migration will not work as expected. Padding will not be updated by <mat-tree> when moving a node inside/outside an expandable node (see #29959).

@poirierlouis poirierlouis added bug Something isn't working ux/ui labels Nov 2, 2024
@poirierlouis poirierlouis self-assigned this Nov 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ux/ui
Projects
None yet
Development

No branches or pull requests

1 participant