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] Fix incorrect panels position when using a toolbar #13474

Merged

Conversation

oukunan
Copy link
Contributor

@oukunan oukunan commented Jun 13, 2024

@zannager zannager added the component: data grid This is the name of the generic UI component, not the React module! label Jun 13, 2024
@oukunan oukunan force-pushed the fix-incorrectly-poppers-positioned-toolbar branch from d374c4f to a60177a Compare June 14, 2024 05:28
role="presentation"
/>
>
<GridPanelAnchor role="presentation" data-id="gridPanelAnchor" />
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @oukunan, nice catch!

Unfortunately, putting GridPanelAnchor here causes an issue when scrolling horizontally:
https://codesandbox.io/p/sandbox/silly-firefly-hmtdfn

Screen.Recording.2024-07-10.at.21.33.45.mov

I'll update the PR with a fix.

@mui-bot
Copy link

mui-bot commented Jul 10, 2024

Deploy preview: https://deploy-preview-13474--material-ui-x.netlify.app/

Generated by 🚫 dangerJS against ecb9b08

@cherniavskii cherniavskii added regression A bug, but worse v7.x labels Jul 10, 2024
@cherniavskii cherniavskii changed the title [DataGrid] fix incorrect poppers positioned when using a toolbar [DataGrid] Fix incorrect panels position when using a toolbar Jul 10, 2024
@cherniavskii cherniavskii enabled auto-merge (squash) July 10, 2024 20:07
@cherniavskii cherniavskii merged commit 863939d into mui:master Jul 10, 2024
17 checks passed
DungTiger pushed a commit to DungTiger/mui-x that referenced this pull request Jul 23, 2024
)

Co-authored-by: Andrew Cherniavskii <andrew.cherniavskii@gmail.com>
thomasmoon pushed a commit to thomasmoon/mui-x that referenced this pull request Sep 9, 2024
)

Co-authored-by: Andrew Cherniavskii <andrew.cherniavskii@gmail.com>
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 v7.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[data grid] filter and columns poppers incorrectly positioned when using a toolbar
4 participants