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
[Transition] Add sx prop type #32025
Conversation
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.
Hello, I think you forgot to run yarn proptypes
to add the sx
props to Slide
's proptypes.
I believe you also need to regenerate the docs using yarn docs:api
too.
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.
The transition components does not support the sx
prop. The sx
prop should be used in the inner component (the component they wrap and add the transition).
If this is not clear, maybe we can re-purpose the PR to update the documentation to state this.
@mnajdova This makes sense, however here is the source of confusion from my end:
Not sure what the best approach would be, but there does seem to be some inconsistency in the way the sx prop is handled with regards to the Collapse and Slide components. Is there a functional difference I am not seeing? |
@cvanem I guess you meant the So in conclusion, I would remove the |
I've added this in #30660, I propose we close this PR and open a new one for removing the |
No description provided.