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

docs(ngAnimate): change .ng-enter to .ng-animate #12202

Closed
wants to merge 1 commit into from

Conversation

yichaowang
Copy link
Contributor

.ng-enter class is not added by animateCss service. .enter and .ng-animate is added. Maybe a fix of adding .ng-enter rather than .enter is more proper.

.ng-enter class is not added by animateCss service. .enter and .ng-animate is added. Maybe a fix of adding .ng-enter rather than .enter is more proper.
@gkalpak
Copy link
Member

gkalpak commented Jun 25, 2015

Indeed it seems that the enter and enter-active classes are added, but according to the docs they should ng-enter and ng-enter-active. @matsko, is this a bug ?

Also, the examples on https://docs.angularjs.org/api/ngAnimate seem rather outdated (and have several types (e.g. missing closing ), height: XYZ instead of height: 'XYZpx' etc).

Demo with the example fixed (although other examples need fixing as well, plus there is the .enter vs .ng-enter issue).

@matsko
Copy link
Contributor

matsko commented Jul 16, 2015

No the CSS code is right, but we forgot to add the property that instructs $animateCss to apply the ng- prefix.

@matsko
Copy link
Contributor

matsko commented Jul 16, 2015

This fix has taken care of that: 344dffb

@matsko
Copy link
Contributor

matsko commented Jul 16, 2015

Thank you for bringing this up.

netman92 pushed a commit to netman92/angular.js that referenced this pull request Aug 8, 2015
ggershoni pushed a commit to ggershoni/angular.js that referenced this pull request Sep 29, 2015
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.

4 participants