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 wrong offset for right-pinned columns when toggling dark/light modes #12233

Merged
merged 1 commit into from Feb 27, 2024

Conversation

cherniavskii
Copy link
Member

@cherniavskii cherniavskii commented Feb 27, 2024

@cherniavskii cherniavskii added bug 🐛 Something doesn't work component: data grid This is the name of the generic UI component, not the React module! plan: Pro Impact at least one Pro user feature: Column pinning Related to the data grid Column pinning feature v7.x labels Feb 27, 2024
@@ -66,7 +66,7 @@ export interface GetHeadersParams {

const SpaceFiller = styled('div')({
/* GridRootStyles conflict */
'&&': {
'&&&': {
Copy link
Member Author

Choose a reason for hiding this comment

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

I don't know why, but style order changes on color mode change, and && is not sufficient anymore:

Screen.Recording.2024-02-27.at.18.51.53.mov

Copy link
Contributor

Choose a reason for hiding this comment

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

That's annoying

@mui-bot
Copy link

mui-bot commented Feb 27, 2024

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

Generated by 🚫 dangerJS against 1ba8cdb

@cherniavskii cherniavskii marked this pull request as ready for review February 27, 2024 18:06
@cherniavskii cherniavskii merged commit 0756108 into mui:next Feb 27, 2024
22 checks passed
@cherniavskii cherniavskii deleted the fix-space-filler-styles branch February 27, 2024 20:14
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: Column pinning Related to the data grid Column pinning feature plan: Pro Impact at least one Pro user v7.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants