-
-
Notifications
You must be signed in to change notification settings - Fork 32.1k
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
[icons-material] Rebuild icons with "use client"
#37894
Conversation
Netlify deploy previewhttps://deploy-preview-37894--material-ui.netlify.app/ Bundle size report |
32e5416
to
8491760
Compare
8491760
to
d18a26e
Compare
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.
I checked only the first file.
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.
Demos work fine and the built icons have the directive. LGTM.
Thanks for checking 🙏 |
Follow up to #37656
This PR rebuilds all the icons with
"use client"
prependedWorking demo: https://codesandbox.io/p/sandbox/condescending-matsumoto-xwtznj?file=%2Fapp%2Fpage.tsx%3A20%2C1