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

Fix issues with ngShow / ngHide animations #9493

Closed
wants to merge 2 commits into from

Conversation

matsko
Copy link
Contributor

@matsko matsko commented Oct 8, 2014

fix($animate): ensure hidden elements with ngShow/ngHide stay hidden during animations

Prior to this fix if an element that contained ng-show or ng-hide was in its hidden state
then any other animation run on the same element would cause the animation to appear despite
the element itself already being hidden. This patch ensures that NO animations are visible
even if the element is set as hidden.

Closes #9103

matsko added 2 commits October 8, 2014 14:36
…during animations

Prior to this fix if an element that contained ng-show or ng-hide was in its hidden state
then any other animation run on the same element would cause the animation to appear despite
the element itself already being hidden. This patch ensures that NO animations are visible
even if the element is set as hidden.

Closes angular#9103
@gkalpak
Copy link
Member

gkalpak commented Oct 8, 2014

NO animations are visible even if the element is set as hidden


AngularJS

The Superheroic JavaScript MVW Framework that hides your animations even if the element is hidden !!!

(pun intended ?)

@matcho : I know what you mean and what this is about (and it's a nice fix), I just thought it sounds funny (by coincidence).

@jeffbcross jeffbcross force-pushed the master branch 2 times, most recently from abdaab7 to 30996f8 Compare October 8, 2014 19:47
@IgorMinar IgorMinar added this to the 1.3.0-rc.5 milestone Oct 8, 2014
@IgorMinar IgorMinar self-assigned this Oct 8, 2014
@matsko matsko closed this in 035ffb8 Oct 8, 2014
matsko added a commit that referenced this pull request Oct 8, 2014
…during animations

Prior to this fix if an element that contained ng-show or ng-hide was in its hidden state
then any other animation run on the same element would cause the animation to appear despite
the element itself already being hidden. This patch ensures that NO animations are visible
even if the element is set as hidden.

Closes #9103
Closes #9493
@btford btford removed the In Progress label Oct 8, 2014
bullgare pushed a commit to bullgare/angular.js that referenced this pull request Oct 9, 2014
bullgare pushed a commit to bullgare/angular.js that referenced this pull request Oct 9, 2014
…during animations

Prior to this fix if an element that contained ng-show or ng-hide was in its hidden state
then any other animation run on the same element would cause the animation to appear despite
the element itself already being hidden. This patch ensures that NO animations are visible
even if the element is set as hidden.

Closes angular#9103
Closes angular#9493
@matsko
Copy link
Contributor Author

matsko commented Oct 10, 2014

Here's the reminder issue: #9549. I will fix this later next week once #8974 goes in.

@matsko matsko deleted the fix_slow_ng_show branch October 10, 2014 12:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ngAnimate late ngShow resolving behavior in v1.3.0-rc.1
5 participants