-
-
Notifications
You must be signed in to change notification settings - Fork 32.2k
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] Migrate Chip demos to emotion #25268
[docs] Migrate Chip demos to emotion #25268
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As a side note. We will need to breakdown the chips demo, to be more focused. There are too convoluted currently.
@oliviertassinari I'll keep it in mind. Right now something like this occurs to me... Basic Chip or Contained Chip
Clickable Chip
Avatar Chip
Icon Chip
Deletable Chip
|
Regarding the breakdown, it feels like the current demos were built for the visual regressions tests, not the developers. We could move them to the test/regressions. For the split, I think that we should aim to have them small enough for the inline preview. I also think that we can do like the the text-field, have the outlined and filled variant documented in the same demo once the notion of variant has been introduced. |
There is an issue associated with this or it is related to it? |
The following demos of the Chip component were migrated:
Related to #16947