Skip to content
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] Add responsive direction and spacing props #26590

Merged
merged 9 commits into from Jun 10, 2021

Conversation

likitarai1
Copy link
Contributor

@likitarai1 likitarai1 commented Jun 4, 2021

The aim of this PR is to have responsive direction and spacing props for Grid component.
Fixes: #6140

Preview: https://deploy-preview-26590--material-ui.netlify.app/components/grid/#responsive-values

@mui-pr-bot
Copy link

mui-pr-bot commented Jun 4, 2021

Details of bundle changes (experimental)

@material-ui/core: parsed: +0.04% , gzip: +0.09%
@material-ui/lab: parsed: +0.06% , gzip: +0.11%

Generated by 🚫 dangerJS against e30ad9b

@oliviertassinari oliviertassinari changed the title Add responsive direction and spacing props [Grid] Add responsive direction and spacing props Jun 6, 2021
@oliviertassinari oliviertassinari added component: Grid The React component. new feature New feature or request labels Jun 6, 2021
@oliviertassinari oliviertassinari force-pushed the direction-spacing-responsive-props branch from 18cb6a6 to a018eed Compare June 6, 2021 14:03
@siriwatknp

This comment has been minimized.

packages/material-ui-system/src/breakpoints.test.js Outdated Show resolved Hide resolved
packages/material-ui/src/Grid/Grid.js Outdated Show resolved Hide resolved
packages/material-ui/src/Stack/Stack.js Outdated Show resolved Hide resolved
@oliviertassinari
Copy link
Member

@siriwatknp Great to have taken the initiative. If we could add a test case for the bug we fix, it would be even better

Copy link
Member

@eps1lon eps1lon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just adding an explicit "request changes" until the other reviews are adressed.

@github-actions github-actions bot added PR: out-of-date The pull request has merge conflicts and can't be merged and removed PR: out-of-date The pull request has merge conflicts and can't be merged labels Jun 7, 2021
@siriwatknp siriwatknp force-pushed the direction-spacing-responsive-props branch from e45ac53 to 382824f Compare June 8, 2021 04:08
@siriwatknp
Copy link
Member

@oliviertassinari I reverted the change I made and resolve conflict from #26559. I think it is good to go (also fix the type columnSpacing: number; rowSpacing: number; to responsive

@siriwatknp siriwatknp merged commit 53de7da into mui:next Jun 10, 2021
@oliviertassinari
Copy link
Member

@siriwatknp Looks great 👍

@oliviertassinari
Copy link
Member

oliviertassinari commented Jun 10, 2021

@siriwatknp We could have updated this list with rowSpacing and columnSpacing

Capture d’écran 2021-06-11 à 00 29 22

https://deploy-preview-26590--material-ui.netlify.app/components/grid/#responsive-values

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: Grid The React component. new feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Grid] Make all the props responsive
5 participants