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/chips): forced colors disabled chip fix #30429

Merged
merged 1 commit into from
Feb 20, 2025

Conversation

DBowen33
Copy link
Contributor

doesn't apply forced color to disabled chips to distingiush from active chips

BEFORE: https://screenshot.googleplex.com/3hAWxwjoxEntW5V
AFTER: https://screenshot.googleplex.com/6wkdMVSXsEmYpPY

fixes b/383354169

doesn't apply forced color to disabled chips to distingiush from active chips

fixes b/383354169
@DBowen33 DBowen33 requested a review from a team as a code owner January 31, 2025 19:23
@DBowen33 DBowen33 requested review from crisbeto and andrewseguin and removed request for a team January 31, 2025 19:23
@andrewseguin andrewseguin added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Feb 19, 2025
@andrewseguin andrewseguin merged commit c5213fd into angular:main Feb 20, 2025
23 of 25 checks passed
@andrewseguin
Copy link
Contributor

The changes were merged into the following branches: main, 19.1.x, 19.2.x

andrewseguin pushed a commit that referenced this pull request Feb 20, 2025
doesn't apply forced color to disabled chips to distingiush from active chips

fixes b/383354169

(cherry picked from commit c5213fd)
andrewseguin pushed a commit that referenced this pull request Feb 20, 2025
doesn't apply forced color to disabled chips to distingiush from active chips

fixes b/383354169

(cherry picked from commit c5213fd)
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: material/chips target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants