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

fix($animate): delegate down to addClass/removeClass if setClass is not found #6476

Closed
wants to merge 1 commit into from

Conversation

matsko
Copy link
Contributor

@matsko matsko commented Feb 27, 2014

Closes #6463

@matsko matsko added cla: yes and removed cla: no labels Feb 28, 2014
//best to catch this early on to prevent any animation operations from occurring
var node = element[0];
if(!node) {
throw new Error();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what's this error about? I read the comment, but if I get this error in my app I'm not going to know what to do about it. Why not use minErr and document it?

@matsko
Copy link
Contributor Author

matsko commented Feb 28, 2014

MERGED

@matsko
Copy link
Contributor Author

matsko commented Feb 28, 2014

Landed as 18c41af

@matsko matsko closed this Feb 28, 2014
@matsko matsko deleted the split_ng_class branch February 28, 2014 06:35
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.

setClass should be broken up into addClass + removeClass JS animations
2 participants