Skip to content

[docs] Migrate Grid demos to emotion#25332

Merged
oliviertassinari merged 17 commits intomui:nextfrom
vicasas:feature/migrate-grid-demo-to-emotion
Mar 14, 2021
Merged

[docs] Migrate Grid demos to emotion#25332
oliviertassinari merged 17 commits intomui:nextfrom
vicasas:feature/migrate-grid-demo-to-emotion

Conversation

@vicasas
Copy link
Member

@vicasas vicasas commented Mar 13, 2021

The following demos of the Grid component were migrated:

  • Spacing
  • Basic grid
  • Grid whit multiple breakpoint
  • Auto Layout
  • Complex Grid
  • Nested Grid
  • Limitations: white-space: nowrap;
  • Interactive Grid
  • CSS Grid

Related to #16947

@vicasas vicasas changed the title Feature/migrate grid demo to emotion [docs] Migrate Grid to emotion Mar 13, 2021
@mui-pr-bot
Copy link

mui-pr-bot commented Mar 13, 2021

No bundle size changes

Generated by 🚫 dangerJS against 1af5f0e

@vicasas
Copy link
Member Author

vicasas commented Mar 13, 2021

For the demos that use Paper component could be used the api styled? thinking of little demos

@oliviertassinari
Copy link
Member

oliviertassinari commented Mar 13, 2021

For the demos that use Paper component could be used the api styled? thinking of little demos

@vicasas Yes please, but only where it can be effective. For instance, I have given it a try in 443a7a3. It's much better. Compare:

Screenshot 2021-03-14 at 00 10 33

Screenshot 2021-03-14 at 00 10 43

@oliviertassinari oliviertassinari added component: Grid The React component. docs Improvements or additions to the documentation. labels Mar 13, 2021
@oliviertassinari oliviertassinari merged commit f7a8681 into mui:next Mar 14, 2021
@oliviertassinari
Copy link
Member

@vicasas Great

@vicasas vicasas changed the title [docs] Migrate Grid to emotion [docs] Migrate Grid demos to emotion Mar 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: Grid The React component. docs Improvements or additions to the documentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants