-
Notifications
You must be signed in to change notification settings - Fork 2.1k
feat(chips): Add mdc-chip-icon-color mixin #2477
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.
Please add this to the demo page.
packages/mdc-chips/README.md
Outdated
@@ -133,7 +133,8 @@ Mixin | Description | |||
`mdc-chip-corner-radius($radius)` | Customizes the corner radius for a chip | |||
`mdc-chip-fill-color-accessible($color)` | Customizes the background fill color for a chip, and updates the chip's ink and ripple color to meet accessibility standards | |||
`mdc-chip-fill-color($color)` | Customizes the background fill color for a chip | |||
`mdc-chip-ink-color($color)` | Customizes the text ink color for a chip, and updates the chip's ripple color to match | |||
`mdc-chip-ink-color($color)` | Customizes the text ink color for a chip, and updates the chip's ripple color to match. This affects both text and icon, unless `mdc-chip-icon-color` is also used. | |||
`mdc-chip-icon-color($color)` | Customizes the icon color for a chip |
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.
Trailing period.
Codecov Report
@@ Coverage Diff @@
## master #2477 +/- ##
=======================================
Coverage 98.91% 98.91%
=======================================
Files 104 104
Lines 4234 4234
Branches 534 534
=======================================
Hits 4188 4188
Misses 46 46 Continue to review full report at Codecov.
|
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.
LGTM
Potentially a fix to #2484, but waiting on designers to confirm if trailing and leading icons should be customizable separately |
No description provided.