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

[data grid] Row ordering does not automatically scroll when trying to drag beyond visible rows #16414

Open
MateuszDev opened this issue Jan 31, 2025 · 1 comment
Labels
component: data grid This is the name of the generic UI component, not the React module! feature: Reordering Related to the data grid Reordering feature plan: Pro Impact at least one Pro user regression A bug, but worse support: premium standard Support request from a Premium standard plan user. https://mui.com/legal/technical-support-sla/

Comments

@MateuszDev
Copy link

MateuszDev commented Jan 31, 2025

Steps to reproduce

Steps:

  1. Open this link to live example: https://codesandbox.io/embed/wl3pg4?module=/src/Demo.tsx&fontsize=12
  2. Grab any row and try to reorder outside the visible rows.
  3. DataGrid does not scroll autmatically
mui-datagrid-v7.mov

Current behavior

DataGrid does not scroll autmatically while reordering

mui-datagrid-v7.mov

Expected behavior

DataGrid does scroll autmatically while reordering like in live example from mui datagrid v6.20.4: https://v6.mui.com/x/react-data-grid/row-ordering/

mui-datagrid-v6.mov

Context

In DataGrid v6.20.4, row reordering works correctly, but in v7.24.1, scrolling stops while reordering. Please restore the behavior from v6.20.4.

Your environment

npx @mui/envinfo
    System:
    OS: macOS 14.4
  Binaries:
    Node: 18.20.5 - ~/.nvm/versions/node/v18.20.5/bin/node
    npm: 10.8.2 - ~/.nvm/versions/node/v18.20.5/bin/npm
    pnpm: Not Found
  Browsers:
    Chrome: 132.0.6834.160
    Edge: Not Found
    Safari: 17.4
  npmPackages:
    @emotion/react: 11.11.1 => 11.11.1 
    @emotion/styled: 11.11.0 => 11.11.0 
    @mui/base: ^5.0.0-beta.58 => 5.0.0-beta.59 
    @mui/core-downloads-tracker:  6.1.4 
    @mui/icons-material: ^6.1.1 => 6.1.4 
    @mui/material: ^6.1.1 => 6.1.4 
    @mui/private-theming:  6.1.4 
    @mui/styled-engine:  6.1.4 
    @mui/system:  6.1.4 
    @mui/types:  7.2.18 
    @mui/utils:  6.1.4 
    @mui/x-data-grid:  7.24.1 
    @mui/x-data-grid-premium: ^7.24.1 => 7.24.1 
    @mui/x-data-grid-pro: ^7.18.0 => 7.24.1 
    @mui/x-date-pickers: ^7.18.0 => 7.20.0 
    @mui/x-date-pickers-pro: ^7.18.0 => 7.20.0 
    @mui/x-internals:  7.20.0 
    @mui/x-license: ^7.18.0 => 7.20.0 
    @types/react: 18.2.33 => 18.2.33 
    react: 18.2.0 => 18.2.0 
    react-dom: 18.2.0 => 18.2.0 
    styled-components: ^5.3.9 => 5.3.11 
    typescript: 5.2.2 => 5.2.2 

Search keywords: data gird row ordering not automatically scroll drag

Order ID: 100890

@MateuszDev MateuszDev added bug 🐛 Something doesn't work status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Jan 31, 2025
@github-actions github-actions bot added component: data grid This is the name of the generic UI component, not the React module! support: premium standard Support request from a Premium standard plan user. https://mui.com/legal/technical-support-sla/ labels Jan 31, 2025
@MateuszDev MateuszDev marked this as a duplicate of #16411 Jan 31, 2025
@KenanYusuf KenanYusuf added regression A bug, but worse plan: Pro Impact at least one Pro user feature: Reordering Related to the data grid Reordering feature and removed bug 🐛 Something doesn't work labels Feb 3, 2025
@github-project-automation github-project-automation bot moved this to 🆕 Needs refinement in MUI X Data Grid Feb 3, 2025
@KenanYusuf KenanYusuf removed the status in MUI X Data Grid Feb 3, 2025
@KenanYusuf KenanYusuf moved this to 🆕 Needs refinement in MUI X Data Grid Feb 3, 2025
@KenanYusuf
Copy link
Member

Thanks for raising this issue @MateuszDev—this indeed look like a regression.

Perhaps we could extend the GridScrollArea to handle scrolling of the virtual scroller for row re-ordering.

@KenanYusuf KenanYusuf removed the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Feb 3, 2025
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! feature: Reordering Related to the data grid Reordering feature plan: Pro Impact at least one Pro user regression A bug, but worse support: premium standard Support request from a Premium standard plan user. https://mui.com/legal/technical-support-sla/
Projects
Status: 🆕 Needs refinement
Development

No branches or pull requests

2 participants