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 reordering does not work with pagination #15351

Open
jackstenglein opened this issue Nov 8, 2024 · 1 comment · May be fixed by #15355
Open

[data grid] Row reordering does not work with pagination #15351

jackstenglein opened this issue Nov 8, 2024 · 1 comment · May be fixed by #15355
Labels
bug 🐛 Something doesn't work 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 support: pro standard Support request from a Pro standard plan user. https://mui.com/legal/technical-support-sla/

Comments

@jackstenglein
Copy link

jackstenglein commented Nov 8, 2024

Steps to reproduce

Link to live example: https://codesandbox.io/p/sandbox/row-reorder-v5r5q9

Steps:

  1. Go to the second page in the DataGrid
  2. Attempt to reorder a row on the second page
  3. The row will instead disappear and cannot be reordered

Current behavior

Rows on pages 2+ cannot be reordered.

Expected behavior

Allow reordering rows on pages other than the first page.

Context

No response

Your environment

npx @mui/envinfo
System:
    OS: macOS 14.4.1
  Binaries:
    Node: 20.18.0 - ~/.nvm/versions/node/v20.18.0/bin/node
    npm: 10.9.0 - ~/.nvm/versions/node/v20.18.0/bin/npm
    pnpm: Not Found
  Browsers:
    Chrome: 130.0.6723.117
    Edge: Not Found
    Safari: 17.4.1
  npmPackages:
    @emotion/react: ^11.13.3 => 11.13.3
    @emotion/styled: ^11.13.0 => 11.13.0
    @mui/base: ^5.0.0-beta.61 => 5.0.0-beta.61
    @mui/core-downloads-tracker:  6.1.6
    @mui/icons-material: ^6.1.6 => 6.1.6
    @mui/lab: ^6.0.0-beta.14 => 6.0.0-beta.14
    @mui/material: ^6.0.1 => 6.1.6
    @mui/material-nextjs: ^6.1.6 => 6.1.6
    @mui/private-theming:  5.16.6
    @mui/styled-engine:  5.16.6
    @mui/system:  5.16.7
    @mui/types:  7.2.19
    @mui/utils:  5.16.6
    @mui/x-charts: ^7.22.1 => 7.22.1
    @mui/x-charts-vendor:  7.20.0
    @mui/x-data-grid:  7.22.2
    @mui/x-data-grid-pro: ^7.22.2 => 7.22.2
    @mui/x-date-pickers: ^7.22.1 => 7.22.1
    @mui/x-internals:  7.21.0
    @mui/x-license:  7.21.0
    @types/react: ^18.3.12 => 18.3.12
    react: ^18.2.0 => 18.3.1
    react-dom: ^18.2.0 => 18.3.1
    typescript: ^5.4.5 => 5.5.4

Used Browser: Firefox 132.0.1 (aarch64)

Search keywords: row,reorder,pagination
Order ID: 101523

@jackstenglein jackstenglein added bug 🐛 Something doesn't work status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Nov 8, 2024
@github-actions github-actions bot added component: data grid This is the name of the generic UI component, not the React module! support: pro standard Support request from a Pro standard plan user. https://mui.com/legal/technical-support-sla/ labels Nov 8, 2024
@k-rajat19 k-rajat19 linked a pull request Nov 9, 2024 that will close this issue
1 task
@michelengelen
Copy link
Member

Thanks @k-rajat19 for opening a PR to fix this! 👍🏼

@michelengelen michelengelen added feature: Reordering Related to the data grid Reordering feature and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Nov 11, 2024
@michelengelen michelengelen changed the title [DataGridPro] Row reordering does not work with pagination [data grid] Row reordering does not work with pagination Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work 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 support: pro standard Support request from a Pro standard plan user. https://mui.com/legal/technical-support-sla/
Projects
Status: 🆕 Needs refinement
Development

Successfully merging a pull request may close this issue.

2 participants