-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
fix(checkbox): support high contrast mode in Firefox on Windows #2927
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2927 +/- ##
======================================
Coverage 98.3% 98.3%
======================================
Files 101 101
Lines 4363 4363
Branches 563 563
======================================
Hits 4289 4289
Misses 74 74 Continue to review full report at Codecov.
|
@@ -69,10 +69,8 @@ | |||
} | |||
|
|||
@mixin mdc-checkbox-ink-color($color) { | |||
.mdc-checkbox__checkmark-path { | |||
// !important is needed here because a stroke must be set as an attribute on the SVG in order |
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.
I did not find this comment being true at all. In fact I've moved stroke color definition from html to css.
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!
(cherry picked from commit 2bd52c7)
NOTE: The
stroke="white"
SVG attribute has been removed as it is no longer needed.