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 missing wrap-reverse classname #9076

Merged
merged 1 commit into from
Nov 9, 2017
Merged

[Grid] Add missing wrap-reverse classname #9076

merged 1 commit into from
Nov 9, 2017

Conversation

dehli
Copy link
Contributor

@dehli dehli commented Nov 9, 2017

Currently the following two grids result in the same component:

<Grid wrap="wrap-reverse" />
<Grid />

This PR allows for the wrap-reverse property to be passed into the Grid component and have the appropriate CSS applied.

Thanks for the great library! 😄

@oliviertassinari oliviertassinari changed the title Add wrap-reverse class to Grid. [Grid] Add missing wrap-reverse classname Nov 9, 2017
@oliviertassinari oliviertassinari added bug 🐛 Something doesn't work component: Grid The React component. PR: accepted labels Nov 9, 2017
@oliviertassinari oliviertassinari merged commit 8f8ef8c into mui:v1-beta Nov 9, 2017
@oliviertassinari
Copy link
Member

@dehli Good catch, thanks for your time!

the-noob pushed a commit to the-noob/material-ui that referenced this pull request Nov 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work component: Grid The React component.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants