Skip to content

Conversation

@danilo-leal
Copy link
Collaborator

@danilo-leal danilo-leal commented May 13, 2023

https://deploy-preview-37259--material-ui.netlify.app/material-ui/discover-more/showcase/

This PR gives a design revamp to the Material UI showcase page. I've mostly switched it to be a 2-column grid instead of just a list so readers can see more items at the same time.

Before After
Screen Shot 2023-05-31 at 12 35 56 Screen Shot 2023-05-31 at 12 36 02

@danilo-leal danilo-leal added docs Improvements or additions to the documentation. package: material-ui labels May 13, 2023
@mui-bot
Copy link

mui-bot commented May 13, 2023

Netlify deploy preview

Bundle size report

No bundle size changes

Generated by 🚫 dangerJS against 73f249c

@danilo-leal danilo-leal self-assigned this May 17, 2023
@danilo-leal danilo-leal marked this pull request as ready for review May 31, 2023 16:01
Copy link
Contributor

@mapache-salvaje mapache-salvaje left a comment

Choose a reason for hiding this comment

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

Looks great! Revisiting this page now I realize how outdated it is so we should definitely give this another refresher for new projects in the very near future.

@danilo-leal danilo-leal merged commit bf1a4fc into master Jun 1, 2023
@danilo-leal danilo-leal deleted the material-ui-showcase-design-revamp branch June 1, 2023 11:05
---

{{"demo": "Showcase.js", "hideToolbar": true, "bg": "inline"}}
{{"component": "docs/src/modules/components/MaterialShowcase.js"}}
Copy link
Member

Choose a reason for hiding this comment

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

This new file location feels wrong. We should be able to render the component at the same location as the markdown. Issue created #42225

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to the documentation.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants