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.

Sign up for GitHub

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

Jump to bottom

TransitionGroup children el may be null #9067

Closed
royalknight56 opened this issue Aug 29, 2023 · 2 comments · Fixed by #9421
Closed

TransitionGroup children el may be null #9067

royalknight56 opened this issue Aug 29, 2023 · 2 comments · Fixed by #9421
Labels
🔨 p3-minor-bug Priority 3: this fixes a bug, but is an edge case that only affects very specific usage. has PR A pull request has already been submitted to solve the issue scope: transition

Comments

@royalknight56
Copy link
Contributor

royalknight56 commented Aug 29, 2023 •
edited
Loading

Vue version

3.2.45

Link to minimal reproduction

https://cv4p7l.csb.app/

Steps to reproduce

click delete button
wait second

What is expected?

no error reported

What is actually happening?

error reported

System Info

No response

Any additional comments?

When the children of TransitionGroup are components, first set the elements inside the component to empty through v-if. Afterwards, an error will be reported when TransitionGroup executes the end animation.

https://codesandbox.io/s/recursing-surf-cv4p7l?file=/src/App.vue
@royalknight56 royalknight56 mentioned this issue Aug 29, 2023
Closed
@royalknight56
Copy link
Contributor Author

#9068

@haoqunjiang haoqunjiang added scope: transition 🔨 p3-minor-bug Priority 3: this fixes a bug, but is an edge case that only affects very specific usage. has PR A pull request has already been submitted to solve the issue labels Aug 29, 2023
@edison1105 edison1105 mentioned this issue Oct 24, 2023
Merged
@edison1105
Copy link
Member

duplicate of #5168

@github-actions github-actions bot locked and limited conversation to collaborators Apr 30, 2024
lynxlangya pushed a commit to lynxlangya/core that referenced this issue May 30, 2024
@edison1105 @lynxlangya
fix(TransitionGroup): avoid set transition hooks for comment nodes an…
64ed0e0 
…d text nodes (vuejs#9421)

close vuejs#4621
close vuejs#4622
close vuejs#5153
close vuejs#5168
close vuejs#7898
close vuejs#9067
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
🔨 p3-minor-bug Priority 3: this fixes a bug, but is an edge case that only affects very specific usage. has PR A pull request has already been submitted to solve the issue scope: transition
Projects
None yet
Milestone
No milestone
3 participants
@haoqunjiang @edison1105 @royalknight56