-
-
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
[system][zero][tag] Add support for sx prop #38535
Conversation
Netlify deploy previewhttps://deploy-preview-38535--material-ui.netlify.app/ Bundle size report |
4904ce0
to
8b990f1
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.
We should look into adding tests next - this way we can replicate most of the system tests in order to be able that all APIs work. It's hard to validate otherwise.
43ddee8
to
e24a255
Compare
User can pass object or a callback that returns an object. This is done through an intermediate babel-plugin that rewrites the object passed to `sx` prop.
e24a255
to
00261dd
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.
All the use-cases we have on the demo seem to work, well done 👍
We should look into a simpler way of how we can test this. Should we have somehow the app deployed on netlify? It takes a lot of time to set up everything in order to review a PR. It's fine for now, because mostly I am the reviewer, but when more people will need to be involved, it is going to waste a lot of time.
Closes #38754 |
User can pass object or a callback that returns an object.