Skip to content
This repository was archived by the owner on Sep 5, 2024. It is now read-only.

interimElement: finish animation of previous element #8624

Closed
devversion opened this issue Jun 2, 2016 · 0 comments · Fixed by #9053
Closed

interimElement: finish animation of previous element #8624

devversion opened this issue Jun 2, 2016 · 0 comments · Fixed by #9053
Assignees
Labels
has: Pull Request A PR has been created to address this issue P2: required Issues that must be fixed. type: bug

Comments

@devversion
Copy link
Member

devversion commented Jun 2, 2016

The $$interimElement factory currently only supports single interimElements at the same time.


This is intentionally, but it doesn't work properly for some components.

This is caused by the $$interimElement's item stack.

Once a second interim element is opening, we cancel the previous interim element and show the new one after the animation has started

This is not correct, because we should wait for the animation to be finished

@devversion devversion added the P2: required Issues that must be fixed. label Jun 2, 2016
@devversion devversion self-assigned this Jun 2, 2016
@ThomasBurleson ThomasBurleson modified the milestone: - Backlog Jun 4, 2016
devversion added a commit to devversion/material that referenced this issue Jul 16, 2016
* Adds support for multiple interim elements (like dialog)
* When single interim (default) is enabled, then interims should hide properly (as in toasts).

Fixes angular#8624. References angular#8630.
@devversion devversion added has: Pull Request A PR has been created to address this issue type: bug labels Jul 16, 2016
devversion added a commit to devversion/material that referenced this issue Sep 1, 2016
* Adds support for multiple interim elements (like dialog)
* When single interim (default) is enabled, then interims should hide properly (as in toasts).

Fixes angular#8624. References angular#8630.
jelbourn pushed a commit that referenced this issue Sep 21, 2016
* feat(interimElement): properly handle multiple interims.

* Adds support for multiple interim elements (like dialog)
* When single interim (default) is enabled, then interims should hide properly (as in toasts).

Fixes #8624. References #8630.

* When hiding wait for currenlty opening interims

* Address EladBezalel's feedback

* Wait for interim element for cancel as well

* Address EladBezalel's feedback
Frank3K pushed a commit to Frank3K/material that referenced this issue Oct 8, 2016
* feat(interimElement): properly handle multiple interims.

* Adds support for multiple interim elements (like dialog)
* When single interim (default) is enabled, then interims should hide properly (as in toasts).

Fixes angular#8624. References angular#8630.

* When hiding wait for currenlty opening interims

* Address EladBezalel's feedback

* Wait for interim element for cancel as well

* Address EladBezalel's feedback
@Splaktar Splaktar removed this from the - Backlog milestone Feb 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
has: Pull Request A PR has been created to address this issue P2: required Issues that must be fixed. type: bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants