Skip to content

Conversation

crisbeto
Copy link
Member

We were setting the snack bar action color to currentColor on dark themes which might not work as expected if there is a color on the body. These changes set it to an explicit color.

Fixes #26247.

We were setting the snack bar action color to `currentColor` on dark themes which might not work as expected if there is a `color` on the `body`. These changes set it to an explicit color.

Fixes angular#26247.
@crisbeto crisbeto added P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent target: patch This PR is targeted for the next patch release labels Dec 15, 2022
@crisbeto crisbeto requested a review from mmalerba December 15, 2022 09:29
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

@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Dec 16, 2022
@crisbeto crisbeto merged commit a6c3f79 into angular:main Dec 16, 2022
crisbeto added a commit that referenced this pull request Dec 16, 2022
…26268)

We were setting the snack bar action color to `currentColor` on dark themes which might not work as expected if there is a `color` on the `body`. These changes set it to an explicit color.

Fixes #26247.

(cherry picked from commit a6c3f79)
crapStone pushed a commit to Calciumdibromid/CaBr2 that referenced this pull request Jan 8, 2023
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [@angular/cdk](https://github.com/angular/components) | dependencies | patch | [`15.0.3` -> `15.0.4`](https://renovatebot.com/diffs/npm/@angular%2fcdk/15.0.3/15.0.4) |
| [@angular/material](https://github.com/angular/components) | dependencies | patch | [`15.0.3` -> `15.0.4`](https://renovatebot.com/diffs/npm/@angular%2fmaterial/15.0.3/15.0.4) |

---

### Release Notes

<details>
<summary>angular/components</summary>

### [`v15.0.4`](https://github.com/angular/components/blob/HEAD/CHANGELOG.md#&#8203;1504-spandex-sword-2023-01-05)

[Compare Source](angular/components@15.0.3...15.0.4)

##### material

| Commit | Type | Description |
| -- | -- | -- |
| [e7fc766379](angular/components@e7fc766) | fix | **menu:** aria-expanded not updating in an OnPush component ([#&#8203;26279](angular/components#26279)) |
| [caa8f7b005](angular/components@caa8f7b) | fix | **menu:** prevent pointer events on animating menu ([#&#8203;26323](angular/components#26323)) |
| [ee63ad32b2](angular/components@ee63ad3) | fix | **slider:** avoid updating tick marks unnecessarily ([#&#8203;26322](angular/components#26322)) |
| [41cfc6e2f4](angular/components@41cfc6e) | fix | **slider:** dragEnd not being emitted ([#&#8203;26289](angular/components#26289)) |
| [6c4a1f74b4](angular/components@6c4a1f7) | fix | **slider:** value jumping after drag with box-sizing: border-box ([#&#8203;26345](angular/components#26345)) |
| [b52cd63bb1](angular/components@b52cd63) | fix | **snack-bar:** incorrect width at some breakpoints ([#&#8203;26334](angular/components#26334)) |
| [5cce2fe6d6](angular/components@5cce2fe) | fix | **snack-bar:** set explicit color on action in dark themes ([#&#8203;26268](angular/components#26268)) |
| [3e50216f6d](angular/components@3e50216) | fix | **tabs:** touch gestures not working on tab nav bar ([#&#8203;26344](angular/components#26344)) |

##### material-experimental

| Commit | Type | Description |
| -- | -- | -- |
| [70cf080cd5](angular/components@70cf080) | fix | **popover-edit:** Make cell focus indicator visible in high contrast (forced colors) mode ([#&#8203;26275](angular/components#26275)) |

#### Special Thanks

Daniel, Karl Seamon, Kristiyan Kostadinov, Paul Gschwendtner and Wagner Maciel

<!-- CHANGELOG SPLIT MARKER -->

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about these updates again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNC44My4xIiwidXBkYXRlZEluVmVyIjoiMzQuODQuMSJ9-->

Co-authored-by: cabr2-bot <[email protected]>
Reviewed-on: https://codeberg.org/Calciumdibromid/CaBr2/pulls/1702
Reviewed-by: Epsilon_02 <[email protected]>
Co-authored-by: Calciumdibromid Bot <[email protected]>
Co-committed-by: Calciumdibromid Bot <[email protected]>
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Jan 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug(snack-bar): wrong snackbar action color with mat-app-background
2 participants