Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(material/snack-bar): snack-bar action button does not follow material guidelines #27769

Merged

Conversation

knoake
Copy link
Contributor

@knoake knoake commented Sep 7, 2023

Fixes a bug in the Angular Material snack-bar, where accent color is used for action button color in light themes and grey is used in dark themes. This does not follow the material design guidelines which recommend a tone of primary color is used in both light and dark themes, and that the color should contrast with the background.

Fixes #13905

@knoake knoake requested a review from mmalerba as a code owner September 7, 2023 11:52
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, very likely to cause screen diffs internally which will need to b resolved

@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 Sep 18, 2023
@knoake knoake force-pushed the snackbar-should-use-primary-color-action-button branch 2 times, most recently from 9b00d0f to d1e3d50 Compare September 28, 2023 13:44
@knoake
Copy link
Contributor Author

knoake commented Sep 28, 2023

Fixed linting errors

@knoake
Copy link
Contributor Author

knoake commented Jan 17, 2024

@mmalerba Is there anything else I need to do to this PR so it can be merged?

@josephperrott josephperrott requested a review from a team as a code owner December 18, 2024 17:40
@josephperrott josephperrott requested review from crisbeto and amysorto and removed request for a team December 18, 2024 17:40
…rial guidelines

Fixes a bug in the Angular Material snack-bar, where accent color is used for action
button color in light themes and grey is used in dark themes. This does not follow the
material design guidelines which recommend a tone of primary color is used in both light
and dark themes, and that the color should contrast with the background.

Fixes angular#13905
@mmalerba mmalerba force-pushed the snackbar-should-use-primary-color-action-button branch from d1e3d50 to 6e61368 Compare February 12, 2025 23:00
@mmalerba mmalerba merged commit 3d7f271 into angular:main Feb 14, 2025
20 of 22 checks passed
@mmalerba
Copy link
Contributor

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

mmalerba pushed a commit that referenced this pull request Feb 14, 2025
…rial guidelines (#27769)

Fixes a bug in the Angular Material snack-bar, where accent color is used for action
button color in light themes and grey is used in dark themes. This does not follow the
material design guidelines which recommend a tone of primary color is used in both light
and dark themes, and that the color should contrast with the background.

Fixes #13905

(cherry picked from commit 3d7f271)
@mmalerba mmalerba self-assigned this Feb 20, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
action: merge The PR is ready for merge by the caretaker area: material/snack-bar target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Snackbar action styling inconsistent / poor contrast
2 participants