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

ngAnimate disrupts correct validation of input type email #12007

Closed
JPtenBerge opened this issue Jun 2, 2015 · 5 comments
Closed

ngAnimate disrupts correct validation of input type email #12007

JPtenBerge opened this issue Jun 2, 2015 · 5 comments

Comments

@JPtenBerge
Copy link

Hi there,

With the newest version (1.4.0) it seems that email validation does not fully work correctly when the ngAnimate module gets included. Check out this Plnkr: http://plnkr.co/edit/mRvFhnpgZCKTI1YPWqkT?p=preview.

Try entering a value like '@@@', '|||' or 'qqq'. The model value only gets updated when a true valid value is entered, but the ng-valid/ng-invalid classes get applied wrongly. Removing the ngAnimate module as a dependency and the ng-valid/invalid classes work just fine.

I'm experiencing this on Windows 7 and 8, in IE as well as Chrome and Firefox.

JP

@Narretz
Copy link
Contributor

Narretz commented Jun 3, 2015

@matsko Wasn't there a similar bug that was already fixed? Or is this the same problem?

@Narretz
Copy link
Contributor

Narretz commented Jun 3, 2015

This seems to be the same issue: #11652

@gkalpak
Copy link
Member

gkalpak commented Jun 3, 2015

It is quite similar, but I don't think it is the same issue, because #11652 has been fixed (demo), but this one is still reproducible on master.

Also, the wrong behaviour is different: This one seems to incorrectly apply the valid classes every 3 keystrokes, even if the value is invalid. Weird and interesting :)

@matsko
Copy link
Contributor

matsko commented Jul 17, 2015

This is now fixed in master courtesy of @sreeramu.

@matsko
Copy link
Contributor

matsko commented Jul 17, 2015

1.4.4 will land next week as well.

netman92 pushed a commit to netman92/angular.js that referenced this issue Aug 8, 2015
…d when cancelled

Instead of merging existing animation option to new animation options we can
merge in reverse and utilize old animation runner.

Closes angular#12266
Closes angular#12007
ggershoni pushed a commit to ggershoni/angular.js that referenced this issue Sep 29, 2015
…d when cancelled

Instead of merging existing animation option to new animation options we can
merge in reverse and utilize old animation runner.

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

No branches or pull requests

5 participants