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

[docs] Breakdown Chip demos into smaller ones #25750

Merged
merged 5 commits into from
Apr 13, 2021

Conversation

vicasas
Copy link
Member

@vicasas vicasas commented Apr 13, 2021

The promised.

Improvements in the Chip component display demos.

Currently, the demos are being wrapped by a Stack component, this should change if the chips texts are longer or if you want to add more chips to the demos. The above so that in mobile it can be seen well.

I await comments so that we continue to iterate the new page proposal.

One chunk of #22484.

Preview: https://deploy-preview-25750--material-ui.netlify.app/components/chips/

@mui-pr-bot
Copy link

mui-pr-bot commented Apr 13, 2021

No bundle size changes

Generated by 🚫 dangerJS against f001a52

@vicasas vicasas marked this pull request as ready for review April 13, 2021 12:35
@oliviertassinari oliviertassinari changed the title [docs] Chip [docs] Breakdown Chip demos into smaller ones Apr 13, 2021
@oliviertassinari oliviertassinari added the docs Improvements or additions to the documentation label Apr 13, 2021
Copy link
Member

@oliviertassinari oliviertassinari left a comment

Choose a reason for hiding this comment

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

It looks almost good to go

docs/src/pages/components/chips/ColorChips.tsx Outdated Show resolved Hide resolved
docs/src/pages/components/chips/SizesChips.tsx Outdated Show resolved Hide resolved
docs/src/pages/components/chips/chips.md Outdated Show resolved Hide resolved
docs/src/pages/components/chips/chips.md Outdated Show resolved Hide resolved
@vicasas
Copy link
Member Author

vicasas commented Apr 13, 2021

@oliviertassinari We delete old demo files?

I remember that it was commented that they should also move to test/regressions but I don't know at what level of the tree.

@oliviertassinari
Copy link
Member

I think that we can remove the old demos and ignore the visual regression. I believe the new demos are enough.

@oliviertassinari oliviertassinari merged commit cce721d into mui:next Apr 13, 2021
@oliviertassinari
Copy link
Member

oliviertassinari commented Apr 13, 2021

Cleaner, well done 👌

@vicasas vicasas deleted the feature/refactor-docs-chips branch April 13, 2021 21:33
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
None yet
Development

Successfully merging this pull request may close these issues.

3 participants