-
-
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
[styled-engine-sc][system] Use namespaced theme prop for css functions in styled-engine-sc #40656
Conversation
Netlify deploy previewhttps://deploy-preview-40656--material-ui.netlify.app/ Bundle size reportDetails of bundle changes (Toolpad) |
c19a06b
to
53adeaa
Compare
53adeaa
to
1b266c5
Compare
I will wait for #40690 before continuing this PR. |
Actually this PR makes #40640 obsolete. My idea was to merge #40640 first (since it contains no breaking changes and is way simpler than this one) and then later this one. I guess this one needs some heavy refactoring after merging #40690? :) Should i try to do it, or do you want to do it? |
Fixes part 2 of #40140
Codesandbox: https://codesandbox.io/p/devbox/fix-styled-engine-sc-css-theme-object-lpkkcv?file=%2Fsrc%2FApp.tsx%3A11%2C1
This PR could probably be considered as a breaking change.