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] Pinned rows are hidden after unpinning #15302

Closed
bwood21 opened this issue Nov 6, 2024 · 2 comments · Fixed by #15697
Closed

[data grid] Pinned rows are hidden after unpinning #15302

bwood21 opened this issue Nov 6, 2024 · 2 comments · Fixed by #15697
Labels
bug 🐛 Something doesn't work component: data grid This is the name of the generic UI component, not the React module! feature: Row pinning Related to the data grid Row pinning feature support: premium standard Support request from a Premium standard plan user. https://mui.com/legal/technical-support-sla/

Comments

@bwood21
Copy link

bwood21 commented Nov 6, 2024

Steps to reproduce

Link to live example. This example is based on the MUI pinning example, but using GridApiRef to set rows instead of using the rows prop.

Steps:

  1. Click 'add row'
  2. Click 'pin at the top' on any row
  3. Click 'unpin' on a pinned row

Current behavior

The unpinned row is hidden

Expected behavior

The unpinned row should be shown in the table as it was before it was pinned

Context

I am trying to pin and unpin rows while using gridApiRef.current.updateRows() - the use case is a large, rapidly updating dataset that has significant performance issues when just using the rows prop to update the table. I am using 7.11.0 in my project, but this issue also occurs on the latest version. This issue is similar to #8013 - I am opening a new issue with an OrderID as requested by MUI X email support.

Your environment

npx @mui/envinfo
I'm running this on WSL and using Chrome from a Windows machine, so the info might be a bit strange here.
System:
    OS: Linux 5.10 Ubuntu 22.04.3 LTS 22.04.3 LTS (Jammy Jellyfish)
  Binaries:
    Node: 22.2.0 - ~/.nvm/versions/node/v22.2.0/bin/node
    npm: 10.7.0 - ~/.nvm/versions/node/v22.2.0/bin/npm
    pnpm: 9.1.0 - ~/.nvm/versions/node/v22.2.0/bin/pnpm
  Browsers:
    Chrome: 130.0.6723.70 (Official Build) (64-bit)

Search keywords: pin pinning gridApiRef
Order ID: 87442

@bwood21 bwood21 added bug 🐛 Something doesn't work status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Nov 6, 2024
@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 Nov 6, 2024
@michelengelen
Copy link
Member

That's definitely weird... I couldn't find a quick solution for this as well.
I'll add it to the board for the team to have a look! 👍🏼

@michelengelen michelengelen changed the title Pinned rows are hidden after unpinning [data grid] Pinned rows are hidden after unpinning Nov 7, 2024
@michelengelen michelengelen removed the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Nov 7, 2024
@github-project-automation github-project-automation bot moved this to 🆕 Needs refinement in MUI X Data Grid Nov 7, 2024
@michelengelen michelengelen added the feature: Row pinning Related to the data grid Row pinning feature label Nov 7, 2024
Copy link

github-actions bot commented Dec 2, 2024

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

@bwood21 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
bug 🐛 Something doesn't work component: data grid This is the name of the generic UI component, not the React module! feature: Row pinning Related to the data grid Row pinning feature support: premium standard Support request from a Premium 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