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

fix(ngAnimate): ensure animations are not attempted on empty jqLite c… #14559

Closed
wants to merge 1 commit into from
Closed

Conversation

maksimr
Copy link
Contributor

@maksimr maksimr commented May 4, 2016

Without this fix animation throw a javascript exception
element.parent not a function when we pass empty jqLite collection.

Closes #14558

…ollection

Without this fix animation throw a javascript exception
`element.parent not a function` when we pass empty jqLite collection.

Closes #14558
@gkalpak gkalpak closed this in 5b053b1 May 6, 2016
gkalpak pushed a commit that referenced this pull request May 6, 2016
Previously `stripCommentsFromElement()` would return an empty Array (instead of a jqLite collection)
which would cause an exception to be thrown: "element.parent not a function".
This commit fixes it, by ensuring that the returned value is always a jqLite collection.

Closes #14558

Closes #14559
gkalpak pushed a commit that referenced this pull request May 13, 2016
Previously `stripCommentsFromElement()` would return an empty Array (instead of a jqLite collection)
which would cause an exception to be thrown: "element.parent not a function".
This commit fixes it, by ensuring that the returned value is always a jqLite collection.

Closes #14558

Closes #14559
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.

(bug) ngAnimate element.parent is not a function for empty jqLite collection
2 participants