Skip to content

fix(experimental/dialog): emitting events twice on some browsers - #13587

Merged
andrewseguin merged 1 commit into
angular:masterfrom
crisbeto:13585/experimental-dialog-duplicate-events
Oct 15, 2018
Merged

fix(experimental/dialog): emitting events twice on some browsers#13587
andrewseguin merged 1 commit into
angular:masterfrom
crisbeto:13585/experimental-dialog-duplicate-events

Conversation

@crisbeto

Copy link
Copy Markdown
Member

Fixes the Dialog emitting some of its events twice on certain browsers due to an issue in @angular/animations which invokes the animation done callback twice.

Fixes#13585.

Fixes the `Dialog` emitting some of its events twice on certain browsers due to an issue in `@angular/animations` which invokes the animation `done` callback twice.
Fixesangular#13585.
@crisbetocrisbeto added pr: merge safe target: patch This PR is targeted for the next patch release labels Oct 12, 2018
@googlebotgooglebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Oct 12, 2018

@jelbournjelbourn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@jelbournjelbourn added pr: lgtm action: merge The PR is ready for merge by the caretaker labels Oct 12, 2018
crisbeto added a commit to crisbeto/material2 that referenced this pull request Oct 14, 2018
Along the same lines as angular#13600 and angular#13587. The `animationDone` will be emitted twice in a row for some browsers due to a bug in `@angular/animations`.
@josephperrott
josephperrott removed their request for review October 15, 2018 14:59
@andrewseguin
andrewseguin merged commit ba7d53d into angular:masterOct 15, 2018
josephperrott pushed a commit that referenced this pull request Oct 19, 2018
…#13608)
Along the same lines as #13600 and #13587. The `animationDone` will be emitted twice in a row for some browsers due to a bug in `@angular/animations`.
josephperrott pushed a commit that referenced this pull request Oct 19, 2018
…#13608)
Along the same lines as #13600 and #13587. The `animationDone` will be emitted twice in a row for some browsers due to a bug in `@angular/animations`.
josephperrott pushed a commit that referenced this pull request Oct 19, 2018
…#13608)
Along the same lines as #13600 and #13587. The `animationDone` will be emitted twice in a row for some browsers due to a bug in `@angular/animations`.
roboshoes pushed a commit to roboshoes/material2 that referenced this pull request Oct 23, 2018
…ular#13587)
Fixes the `Dialog` emitting some of its events twice on certain browsers due to an issue in `@angular/animations` which invokes the animation `done` callback twice.
Fixesangular#13585.
@angular-automatic-lock-bot

Copy link
Copy Markdown

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-botangular-automatic-lock-botBot locked and limited conversation to collaborators Sep 9, 2019
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: mergeThe PR is ready for merge by the caretakercla: yesPR author has agreed to Google's Contributor License Agreementtarget: patchThis PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CdkDialog afterClosed called twice on safari & edge

5 participants

@crisbeto@jelbourn@googlebot@josephperrott@andrewseguin