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
[Grid] Prevent crash if spacing is set to zero in theme #33777
[Grid] Prevent crash if spacing is set to zero in theme #33777
Conversation
Can you review this PR please ? |
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.
@PunitSoniME Thanks for the contribution. See my comment #33771 (comment) for better understanding as to why it is happening.
We will also need to add a test in Grid (Check the CodeSandbox in the above linked comment).
density tool
crashing
density tool
crashing
@siriwatknp @PunitSoniME I have added a test and updated the PR description explaining the issue in #33771 (comment). Please take a look. |
I am changing the PR title to Grid instead. Ideally, this PR should be split into 2 @PunitSoniME thanks for fixing it. |
Thanks @ZeeshanTamboli Thanks @siriwatknp |
This comment was marked as spam.
This comment was marked as spam.
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.
👏 Thank you so much @PunitSoniME @ZeeshanTamboli
* [docs] demo density tool crashes * [docs] prettier executed * [docs] code changed as per suggestions * add test Co-authored-by: ZeeshanTamboli <zeeshan.tamboli@gmail.com>
Resolves #33771
Preview - https://deploy-preview-33777--material-ui.netlify.app/material-ui/customization/density/
DensityTool
demo which uses the Grid from crashing thus resolving [docs] Demo DensityTool crashes #33771.More details in #33771 (comment).
Before: https://codesandbox.io/s/admiring-dirac-ctu7c1.
After: https://codesandbox.io/s/boring-agnesi-l7fmw1