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 columns are broken in RTL mode #14245

Closed
TheWeirdDev opened this issue Aug 17, 2024 · 3 comments · Fixed by #14586
Closed

[data grid] Pinned columns are broken in RTL mode #14245

TheWeirdDev opened this issue Aug 17, 2024 · 3 comments · Fixed by #14586
Assignees
Labels
component: data grid This is the name of the generic UI component, not the React module! feature: Column pinning Related to the data grid Column pinning feature l10n localization plan: Pro Impact at least one Pro user regression A bug, but worse

Comments

@TheWeirdDev
Copy link

TheWeirdDev commented Aug 17, 2024

Steps to reproduce

Link to live example: The example in the docs is also broken

Steps:

  1. Create a data grid with pinned column(s)
  2. Set the theme direction to RTL
  3. Observe pinned column(s) do not work

Current behavior

The pinned column does not stick to the side and some columns become invisible.
This video demonstrates the bug:
Screencast from 2024-08-17 13-41-52.webm

Expected behavior

Pinned columns in RTL should work like they do in LTR direction.

Context

No response

Your environment

System:
OS: Linux 6.10 Arch Linux
Binaries:
Node: 22.6.0 - /usr/bin/node
npm: 10.8.2 - /usr/bin/npm
pnpm: 9.7.1 - /usr/bin/pnpm
Browsers:
Chrome: Not Found
npmPackages:
@emotion/react: ^11.13.0 => 11.13.0
@emotion/styled: ^11.13.0 => 11.13.0
@mui/icons-material: ^5.16.7 => 5.16.7
@mui/lab: 5.0.0-alpha.169 => 5.0.0-alpha.169
@mui/material: ^5.16.7 => 5.16.7
@mui/styles: ^5.16.7 => 5.16.7
@mui/x-data-grid-premium: ^7.12.1 => 7.12.1
@mui/x-data-grid-pro: ^7.12.1 => 7.12.1
@mui/x-date-pickers: ^7.12.1 => 7.12.1
@mui/x-date-pickers-pro: ^7.12.1 => 7.12.1
@mui/x-license: ^7.13.0 => 7.13.0
@mui/x-tree-view: ^7.12.1 => 7.12.1
@types/react: ^18.3.3 => 18.3.3
react: ^18.3.1 => 18.3.1
react-dom: ^18.3.1 => 18.3.1
styled-components: ^6.1.12 => 6.1.12
typescript: ~5.3.3 => 5.3.3

Search keywords: pinned columns,pinned,rtl

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

Thanks @TheWeirdDev for raising this.
I can confirm this bug and will add it to the board. 👍🏼

@michelengelen michelengelen changed the title [Data Grid] Pinned columns are broken in RTL mode [data grid] Pinned columns are broken in RTL mode Aug 19, 2024
@michelengelen michelengelen added l10n localization and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Aug 19, 2024
@rotembarsela
Copy link
Contributor

I had the same problem today when I tried to implement that on an RTL website

@KenanYusuf KenanYusuf self-assigned this Sep 11, 2024
@cherniavskii cherniavskii added plan: Pro Impact at least one Pro user regression A bug, but worse feature: Column pinning Related to the data grid Column pinning feature and removed bug 🐛 Something doesn't work labels Sep 13, 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

We value your feedback @TheWeirdDev! How was your experience with our support team?
If you could spare a moment, we'd love to hear your thoughts in this brief Support Satisfaction survey. Your insights help us improve!

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: Column pinning Related to the data grid Column pinning feature l10n localization plan: Pro Impact at least one Pro user regression A bug, but worse
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

5 participants