Skip to content

feat(material/dialog): add the ability to control the animation duration - #13466

Merged
crisbeto merged 1 commit into
angular:masterfrom
crisbeto:3616/dialog-animation-duration
Mar 14, 2022
Merged

feat(material/dialog): add the ability to control the animation duration#13466
crisbeto merged 1 commit into
angular:masterfrom
crisbeto:3616/dialog-animation-duration

Conversation

@crisbeto

Copy link
Copy Markdown
Member

Since the dialog animation is on the MatDialogContainer, consumers aren't able to disable the animation. These changes add properties to the dialog config that allow consumers to set the duration of the dialog's enter and exit animations.

Fixes#3616.

@crisbetocrisbeto added the target: minor This PR is targeted for the next minor release label Oct 6, 2018
@googlebotgooglebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Oct 6, 2018
@crisbeto
crisbetoforce-pushed the 3616/dialog-animation-duration branch from a8e5e31 to 1dead0fCompareOctober 6, 2018 08:47

@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 8, 2018
crisbeto added a commit to crisbeto/material2 that referenced this pull request Oct 19, 2018
…uration
For keeping parity with the Material dialog (angular#13466), adds the ability for consumers to configure the animation duration of the dialog.
@josephperrottjosephperrott 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 Oct 19, 2018
@josephperrott

Copy link
Copy Markdown
Member

Caretaker Note: Should be merged with #13694 as they are paired together.

@ngbot

ngbotBot commented Nov 3, 2018

Copy link
Copy Markdown

Hi @crisbeto! This PR has merge conflicts due to recent upstream merges.
Please help to unblock it by resolving these conflicts. Thanks!

vivian-hu-zz pushed a commit that referenced this pull request Nov 10, 2018
…uration (#13694)
For keeping parity with the Material dialog (#13466), adds the ability for consumers to configure the animation duration of the dialog.
@crisbeto
crisbetoforce-pushed the 3616/dialog-animation-duration branch from 1dead0f to b933fc1CompareNovember 17, 2018 14:03
@mmalerbammalerba added aaa and removed aaa labels Apr 25, 2019
@andrewseguinandrewseguin added the P5 The team acknowledges the request but does not plan to address it, it remains open for discussion label May 30, 2019
@crisbeto
crisbetoforce-pushed the 3616/dialog-animation-duration branch from b933fc1 to 8735035CompareJune 10, 2019 19:49
@crisbeto
crisbetoforce-pushed the 3616/dialog-animation-duration branch 2 times, most recently from 15a1d63 to 1cb94b5CompareSeptember 22, 2019 17:28
@crisbeto
crisbetoforce-pushed the 3616/dialog-animation-duration branch 2 times, most recently from bea7474 to e0cd1acCompareDecember 25, 2019 14:25
@crisbeto
crisbetoforce-pushed the 3616/dialog-animation-duration branch from e0cd1ac to c678d2dCompareFebruary 25, 2020 21:43
@mmalerbammalerba removed the lgtm label Jul 31, 2020
@liesahead

Copy link
Copy Markdown

Is this is going to be merged?

@lansana

Copy link
Copy Markdown

+1 would love to see this merged as well

@crisbetocrisbeto removed the merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note label Nov 9, 2021
@crisbetocrisbeto changed the title feat(dialog): add the ability to control the animation durationfeat(material/dialog): add the ability to control the animation durationNov 9, 2021
@crisbetocrisbeto added P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent and removed P5 The team acknowledges the request but does not plan to address it, it remains open for discussion labels Nov 9, 2021
@crisbeto
crisbetoforce-pushed the 3616/dialog-animation-duration branch from c678d2d to 7b8856aCompareNovember 9, 2021 08:22
@crisbeto
crisbeto requested a review from a teamNovember 9, 2021 08:22
@crisbeto
crisbetoforce-pushed the 3616/dialog-animation-duration branch 3 times, most recently from 209fff9 to 28465beCompareNovember 10, 2021 11:59
@devversion
devversion removed the request for review from a teamNovember 10, 2021 17:15
@andrewseguinandrewseguin removed the cla: yes PR author has agreed to Google's Contributor License Agreement label Dec 29, 2021
@crisbeto
crisbetoforce-pushed the 3616/dialog-animation-duration branch 2 times, most recently from 36aeca8 to 0b31649CompareMarch 12, 2022 10:08
Since the dialog animation is on the `MatDialogContainer`, consumers aren't able to disable the animation. These changes add properties to the dialog config that allow consumers to set the duration of the dialog's enter and exit animations.
Fixesangular#3616.
@crisbeto
crisbetoforce-pushed the 3616/dialog-animation-duration branch from 0b31649 to a6cea11CompareMarch 12, 2022 10:37
@crisbeto
crisbeto merged commit 8ef3125 into angular:masterMar 14, 2022
forsti0506 pushed a commit to forsti0506/components that referenced this pull request Apr 3, 2022
…ion (angular#13466)
Since the dialog animation is on the `MatDialogContainer`, consumers aren't able to disable the animation. These changes add properties to the dialog config that allow consumers to set the duration of the dialog's enter and exit animations.
Fixesangular#3616.
@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 Apr 14, 2022
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 caretakerP3An issue that is relevant to core functions, but does not impede progress. Important, but not urgenttarget: minorThis PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add possibility to turn off animation of dialog enter/exit

9 participants

@crisbeto@josephperrott@liesahead@lansana@jelbourn@googlebot@zarend@mmalerba@andrewseguin