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-experimental/mdc-snack-bar): avoid multiple snack bars on the page if opened in quick succession #24757

Merged
merged 1 commit into from
May 4, 2022

Conversation

crisbeto
Copy link
Member

@crisbeto crisbeto commented Apr 10, 2022

Fixes an issue where opening a snack bar while the previous one was being animated caused the former to remain in the DOM. The problem was that MDC always waits for an animation, even if it is interrupted.

Caretaker note: Fixes internal issue b/225379279

@crisbeto crisbeto added P2 The issue is important to a large percentage of users, with a workaround G This is is related to a Google internal issue target: patch This PR is targeted for the next patch release labels Apr 10, 2022
@crisbeto crisbeto requested a review from andrewseguin as a code owner April 10, 2022 15:11
@crisbeto crisbeto force-pushed the mdc-snack-bar-multiple branch 3 times, most recently from d505af4 to 5483913 Compare April 10, 2022 15:43
andrewseguin
andrewseguin previously approved these changes Apr 11, 2022
@andrewseguin andrewseguin added the action: merge The PR is ready for merge by the caretaker label Apr 11, 2022
@andrewseguin andrewseguin added the merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note label Apr 22, 2022
@crisbeto crisbeto force-pushed the mdc-snack-bar-multiple branch from 5483913 to 66d1d52 Compare April 26, 2022 12:50
…n the page if opened in quick succession

Fixes an issue where opening a snack bar while the previous one was being animated caused the former to remain in the DOM. The problem was that MDC always waits for an animation, even if it is interrupted.
@crisbeto crisbeto merged commit 30f5181 into angular:main May 4, 2022
crisbeto added a commit that referenced this pull request May 4, 2022
…n the page if opened in quick succession (#24757)

Fixes an issue where opening a snack bar while the previous one was being animated caused the former to remain in the DOM. The problem was that MDC always waits for an animation, even if it is interrupted.

(cherry picked from commit 30f5181)
crapStone pushed a commit to Calciumdibromid/CaBr2 that referenced this pull request May 13, 2022
This PR contains the following updates:

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

---

### Release Notes

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

### [`v13.3.6`](https://github.com/angular/components/blob/HEAD/CHANGELOG.md#&#8203;1336-iridium-ire-2022-05-04)

[Compare Source](angular/components@13.3.5...13.3.6)

##### material

| Commit | Type | Description |
| -- | -- | -- |
| [6a0e6ba510](angular/components@6a0e6ba) | fix | **menu:** add constructor overloads for non-deprecated signatures ([#&#8203;24860](angular/components#24860)) |

##### material-experimental

| Commit | Type | Description |
| -- | -- | -- |
| [8ca629dc7e](angular/components@8ca629d) | fix | **mdc-snack-bar:** avoid multiple snack bars on the page if opened in quick succession ([#&#8203;24757](angular/components#24757)) |
| [d6f9db37b0](angular/components@d6f9db3) | fix | **mdc-table:** add inherit background to rows ([#&#8203;24867](angular/components#24867)) |

#### Special Thanks

Andrew Seguin, Daniil Dubrava, Jeremy Elbourn, Kristiyan Kostadinov and Paul Gschwendtner

<!-- CHANGELOG SPLIT MARKER -->

</details>

---

### Configuration

📅 **Schedule**: 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, click this checkbox.

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).

Co-authored-by: cabr2-bot <[email protected]>
Reviewed-on: https://codeberg.org/Calciumdibromid/CaBr2/pulls/1342
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 Jun 4, 2022
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 G This is is related to a Google internal issue merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note P2 The issue is important to a large percentage of users, with a workaround 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