Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

fix($animate): use a scheduled timeout in favor of a fallback property to close transitions #5403

Closed
wants to merge 2 commits into from

Conversation

matsko
Copy link
Contributor

@matsko matsko commented Dec 14, 2013

With ngAnimate, CSS transitions, that are not properlty triggered, are forceably closed off
by appling a fallback property. The fallback property approach works, however, its styling
itself may effect CSS inheritance or cause the element to render improperly. Therefore, its
best to stick to using a scheduled timeout to run sometime after the highest animation time
has passed.

Closes #5255

@scottopherson
Copy link

Nice, all my animations work great with this fix 👍

…y to close transitions

With ngAnimate, CSS transitions, that are not properlty triggered, are forceably closed off
by appling a fallback property. The fallback property approach works, however, its styling
itself may effect CSS inheritance or cause the element to render improperly. Therefore, its
best to stick to using a scheduled timeout to run sometime after the highest animation time
has passed.

Closes angular#5255
Closes angular#5241
Closes angular#5405
@matsko
Copy link
Contributor Author

matsko commented Dec 19, 2013

MERGED

@matsko matsko closed this Dec 19, 2013
@matsko
Copy link
Contributor Author

matsko commented Dec 19, 2013

Landed as 54637a3

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ng-animate border-spacing causes slight jump when animating
3 participants