Skip to content
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(material/checkbox): set checkbox checkmark color with default-con… #27381

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

stephenrca
Copy link

…trast

Fixes a bug in the Angular Material checkbox component where checkmark does not have the right color if theme is set with CSS variables.

Fixes #27204

@google-cla
Copy link

google-cla bot commented Jun 28, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@amysorto amysorto self-assigned this Jun 30, 2023
@amysorto
Copy link
Contributor

Looks like there are some lint and test failures. Can you look into it @stephenrca?

@stephenrca
Copy link
Author

stephenrca commented Jul 3, 2023 via email

Copy link
Contributor

@mmalerba mmalerba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mmalerba mmalerba added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Aug 1, 2023
@stephenrca stephenrca force-pushed the stephenrca-fix-checkbox-checkmark-color branch 3 times, most recently from 34413e0 to 6bd8b18 Compare August 2, 2023 08:13
@angular-robot angular-robot bot added detected: feature PR contains a feature commit area: docs Related to the documentation area: build & ci Related the build and CI infrastructure of the project labels Aug 2, 2023
aanchal88 and others added 2 commits August 2, 2023 10:17
* docs: Revised the HTML tags

* update

* update

* update

* update
…trast

Fixes a bug in the Angular Material `checkbox` component where checkmark does not have the right color if theme is set with CSS variables.

Fixes angular#27204
@stephenrca stephenrca force-pushed the stephenrca-fix-checkbox-checkmark-color branch from 6bd8b18 to 5024837 Compare August 2, 2023 08:18
@angular-robot angular-robot bot removed detected: feature PR contains a feature commit area: build & ci Related the build and CI infrastructure of the project labels Aug 2, 2023
@devversion devversion removed their request for review August 7, 2023 12:44
@josephperrott josephperrott removed their request for review October 3, 2023 14:47
@tinesoft
Copy link

tinesoft commented Jan 5, 2024

Hi @stephenrca, @mmalerba
Any progress on this PR?
It would be good is it could be backported to Angular Material 16 as well...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: merge The PR is ready for merge by the caretaker area: docs Related to the documentation target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wrong checkbox checkmark color when palettes are set with CSS variables
6 participants