-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
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
[docs][material-ui][system] Edit "How to use components selector API" section to add about direct import #43377
Conversation
Netlify deploy previewBundle size report |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@IAluI It looks good. But can you open a new PR and point it to master
instead? Version 6 also supports Emotion JS. You can tag me once you open a PR and close this one.
- add @mui/material to @emotion/babel-plugin config - add note about @emotion/babel-plugin and babel-plugin-direct-import order Signed-off-by: Ilya Molostov <29124424+IAluI@users.noreply.github.com>
@ZeeshanTamboli I noticed that MUI changed the default branch. I made the appropriate changes to my fork. Still need a new PR? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@IAluI Looks good! I've linked proof that it works here: https://stackblitz.com/edit/vitejs-vite-t82o8s?file=vite.config.ts,src%2FApp.tsx&terminal=dev (also added in the description). I made a few tweaks, which you can see in the commits. Thanks for the pull request!
Proof that it works: https://stackblitz.com/edit/vitejs-vite-t82o8s?file=vite.config.ts,src%2FApp.tsx&terminal=dev