-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2099 +/- ##
=======================================
Coverage 99.33% 99.33%
=======================================
Files 84 84
Lines 3769 3769
Branches 490 490
=======================================
Hits 3744 3744
Misses 25 25 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.
Remove references to "dark mode" and mdc-theme--dark
from README.md.
packages/mdc-slider/mdc-slider.scss
Outdated
@include mdc-slider-highlight-color_($disabled-color); | ||
@include mdc-slider-rail-color_($disabled-color); | ||
@include mdc-slider-rail-tick-mark-color_($disabled-color); | ||
@include mdc-slider-thumb-color_($disabled-color); | ||
@include mdc-slider-thumb-stroke-cutout_($mdc-slider-default-assumed-bg-color); |
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.
We can remove the $mdc-slider-dark-theme-assumed-bg-color
variable from _variables.scss
Don't forget to add BREAKING CHANGE
to the PR description and commit message
BREAKING CHANGE: removed `$mdc-slider-dark-theme-assumed-bg-color` from slider variables.
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!
No description provided.