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

1.2.6 ng-animate: doesn't transition when a 2nd transition is triggered #5588

Closed
bbottema opened this issue Jan 1, 2014 · 4 comments
Closed
Assignees
Milestone

Comments

@bbottema
Copy link

bbottema commented Jan 1, 2014

jsfiddle

Angularjs 1.2.6 scenario: 2 inputs, 1 textbox (sounds kinda dirty). The inputs both trigger a show / hide transition on blur / focus events and focus always comes after blur if click from one input to another. Even the text binding shows the proper current state (info.show = true or false based on these focus / blur events).

Without ngAnimate it works perfect. When ngAnimate is added as dependency, transitions stop working properly. In the jsfiddle, simply remove ngAnimate (leaving an empty array) and it works just fine.

Just click on one input: textbox fades in. Now directly click the other input: text updates properly, but box still fades out.

@matsko
Copy link
Contributor

matsko commented Jan 6, 2014

I've been working on this for the past few days. This wonderous example introduced three separate bugs into the addClass / removeClass feature. On bug three now, almost there.

matsko added a commit to matsko/angular.js that referenced this issue Jan 6, 2014
@matsko
Copy link
Contributor

matsko commented Jan 6, 2014

@matsko
Copy link
Contributor

matsko commented Jan 6, 2014

That was a massive challenge. Thanks a bunch for pointing out this bug.

matsko added a commit to matsko/angular.js that referenced this issue Jan 6, 2014
@bbottema
Copy link
Author

bbottema commented Jan 6, 2014

Happy to see it fixed and I'm glad I could be of some help.

@bbottema bbottema closed this as completed Jan 6, 2014
matsko added a commit to matsko/angular.js that referenced this issue Jan 14, 2014
IgorMinar pushed a commit to IgorMinar/angular.js that referenced this issue Jan 14, 2014
matsko added a commit to matsko/angular.js that referenced this issue Jan 14, 2014
matsko added a commit to matsko/angular.js that referenced this issue Jan 14, 2014
jamesdaily pushed a commit to jamesdaily/angular.js that referenced this issue Jan 27, 2014
jamesdaily pushed a commit to jamesdaily/angular.js that referenced this issue Jan 27, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants