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

[DataGrid] Scroll starts jumping back up when holding down arrow keys #15158

Closed
lauri865 opened this issue Oct 28, 2024 · 3 comments · Fixed by #15164
Closed

[DataGrid] Scroll starts jumping back up when holding down arrow keys #15158

lauri865 opened this issue Oct 28, 2024 · 3 comments · Fixed by #15164
Assignees
Labels
component: data grid This is the name of the generic UI component, not the React module! regression A bug, but worse

Comments

@lauri865
Copy link
Contributor

lauri865 commented Oct 28, 2024

Steps to reproduce

Link to live example: (required)

Steps:

  1. https://mui.com/x/
  2. Hold down arrow key and wait a bit
arrow-navigation-lag.mp4

Current behavior

Scroll index jumps back up

Expected behavior

Scroll continues smoothly and focused column should stay as the last one

Context

Seems like a recent regression. As a side-nitpick - the last row position is imprecise even when it doesn't jump – sometimes up by a pixel, sometimes down.

Your environment

npx @mui/envinfo
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Search keywords: arrow keys navigation jump

@lauri865 lauri865 added bug 🐛 Something doesn't work status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Oct 28, 2024
@github-actions github-actions bot added the component: data grid This is the name of the generic UI component, not the React module! label Oct 28, 2024
@lauri865
Copy link
Contributor Author

lauri865 commented Oct 28, 2024

First introduced in v7.20.0

Probably #14888 @arminmeh?

I believe regular scroll performance is also affected.

And fwiw, it's not particular to any grouping, etc. We first noticed it with a pretty plain setup - just 2 pinned columns. But it doesn't reproduce immediately in this demo: https://mui.com/x/react-data-grid/#pro-plan

@arminmeh arminmeh self-assigned this Oct 29, 2024
@arminmeh
Copy link
Contributor

@lauri865 thanks for reporting.
I have confirmed that this is a regression and will work on the fix

@arminmeh arminmeh added regression A bug, but worse and removed bug 🐛 Something doesn't work status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Oct 29, 2024
Copy link

This issue has been closed. If you have a similar problem but not exactly the same, please open a new issue.
Now, if you have additional information related to this issue or things that could help future readers, feel free to leave a comment.

Note

@lauri865 How did we do? Your experience with our support team matters to us. If you have a moment, please share your thoughts in this short Support Satisfaction survey.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: data grid This is the name of the generic UI component, not the React module! regression A bug, but worse
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants