Skip to content

[docs] Add CSS bundler breaking change to migration guide #17436

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

romgrk
Copy link
Contributor

@romgrk romgrk commented Apr 18, 2025

Follow-up of #17427

@romgrk romgrk added docs Improvements or additions to the documentation enhancement This is not a bug, nor a new feature labels Apr 18, 2025
@romgrk romgrk requested a review from a team April 18, 2025 10:03
@mui-bot
Copy link

mui-bot commented Apr 18, 2025

### Bundling

The Data Grid now requires a bundler that can handle CSS imports.

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

What level of detail do you have in mind? Just a link?

Copy link
Member

Choose a reason for hiding this comment

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

Something like https://mui.com/x/migration/usage-with-material-ui-v5-v6/, with code snippets and links

Copy link
Member

@oliviertassinari oliviertassinari left a comment

Choose a reason for hiding this comment

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

To what I understand Joy UI has the same requirements. So let's have Core review on this one. X can be responsible to fix this but Core should be accountable for this.

@cherniavskii
Copy link
Member

cc @DiegoAndai

@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Apr 24, 2025
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to the documentation enhancement This is not a bug, nor a new feature PR: out-of-date The pull request has merge conflicts and can't be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants