Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

feat(modal): Switched the modal transclude to use $animate #6029

Closed
wants to merge 1 commit into from
Closed

feat(modal): Switched the modal transclude to use $animate #6029

wants to merge 1 commit into from

Conversation

cleavera
Copy link

Modal transclude switched to use $animate.enter rather than element.append, this is so that the transcluded content can hook in the animation lifecycle. Also added unit tests for the uibModalTransclude directive as there were none before.

Closes #6023

Modal transclude now uses $animate.enter rather than element.append, this is so that the transcluded content can hook in the animation lifecycle. Also added unit tests for the uibModalTransclude directive as there were none before.

Closes #6023
@wesleycho
Copy link
Contributor

LGTM

@wesleycho wesleycho added this to the 2.0.0 milestone Jun 27, 2016
@wesleycho wesleycho closed this in 1cbd73d Jun 27, 2016
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.

2 participants