Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test(TransitionGroup): test for TransitionGroup #1269

Merged
merged 6 commits into from
Jun 25, 2020

Conversation

underfin
Copy link
Member

No description provided.

transitionNumber--
if (transitionNumber === 0) {
;[enterFromClass, enterActiveClass, enterToClass] = originEnterClass
}
Copy link
Member Author

Choose a reason for hiding this comment

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

For TransitionGroup has more than one child nodes. Need reset class when last node transition end.
This solution can be work.Maybe has more better solution, but I just has this one.

<!-- return { toggle, click }-->
<!-- }-->
<!-- }).mount('#app')-->
<!--</script>-->
Copy link
Member Author

@underfin underfin May 31, 2020

Choose a reason for hiding this comment

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

This has confict with #1151

packages/runtime-dom/src/components/TransitionGroup.ts Outdated Show resolved Hide resolved
instance.vnode.el as Node,
moveClass
)
) {
Copy link
Member Author

@underfin underfin May 31, 2020

Choose a reason for hiding this comment

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

This need check when every update triggle.Because maybe has dynamic name which has css transform.
Well. we can reset value of hasMove when re-invoke render with returned by setup.But this can be more directly and clearly.

@yyx990803 yyx990803 force-pushed the transition-group branch 2 times, most recently from c78a193 to 9a2456f Compare June 25, 2020 18:19
@yyx990803 yyx990803 merged commit 299fda4 into vuejs:master Jun 25, 2020
@underfin underfin deleted the transition-group branch June 26, 2020 01:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants