You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
A gap to fill as the Material You specs do not specify how to use other colors for the chip component. How it works right now is similar to v5:
If no color prop is provided, then the specs defined color is used:
outline color for the outlined variant
secondary color for the filled variant
surface container low color for the elevated variant
If the color prop is provided, that one is used. Right now this only supports accent colors.
Points of improvement:
Feels weird that when no color prop is provided, different colors are used for different variants. This is the less confusing way I could come up with on initial implementation but I think there should be a better way
Supporting other colors? The difficult part here is that not all variants should support all colors
Supporting other surface container colors might be useful for the elevated variant
Supporting outline variant color might be useful for the outlined variant
Adding a "high-contrast" option which is similar to v5 might be useful for the filled and elevated variants, e.g. using the primary and onPrimary tokens instead of the primaryContainer and onPrimaryContainer colors
Restricting colors? The color specs mention that chips should use only secondary color. That feels restrictive, but maybe an option we want to explore?
The text was updated successfully, but these errors were encountered:
A gap to fill as the Material You specs do not specify how to use other colors for the chip component. How it works right now is similar to v5:
color
prop is provided, then the specs defined color is used:color
prop is provided, that one is used. Right now this only supports accent colors.Points of improvement:
color
prop is provided, different colors are used for different variants. This is the less confusing way I could come up with on initial implementation but I think there should be a better wayprimary
andonPrimary
tokens instead of theprimaryContainer
andonPrimaryContainer
colorsThe text was updated successfully, but these errors were encountered: