fix: fix transition-group processing of whitespace nodes

catrope · catrope · commit bef207dc8f38 · 2021-09-21T15:44:46.000-07:00
fixes vuejs#4621, vuejs#4622
diff --git a/packages/runtime-core/src/components/BaseTransition.ts b/packages/runtime-core/src/components/BaseTransition.ts
@@ -8,6 +8,7 @@ import {
   cloneVNode,
   Comment,
   isSameVNodeType,
+  Text,
   VNode,
   VNodeArrayChildren,
   Fragment
@@ -470,7 +471,8 @@ export function getTransitionRawChildren(
       )
     }
     // comment placeholders should be skipped, e.g. v-if
-    else if (keepComment || child.type !== Comment) {
+    // #4621, #4622, #4637 also skip text nodes
+    else if ((keepComment || child.type !== Comment) && child.type !== Text) {
       ret.push(child)
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -8,6 +8,7 @@ import {`
`8`	`8`	`cloneVNode,`
`9`	`9`	`Comment,`
`10`	`10`	`isSameVNodeType,`
	`11`	`+ Text,`
`11`	`12`	`VNode,`
`12`	`13`	`VNodeArrayChildren,`
`13`	`14`	`Fragment`
`@@ -470,7 +471,8 @@ export function getTransitionRawChildren(`
`470`	`471`	`)`
`471`	`472`	`}`
`472`	`473`	`// comment placeholders should be skipped, e.g. v-if`
`473`		`- else if (keepComment \|\| child.type !== Comment) {`
	`474`	`+ // #4621, #4622, #4637 also skip text nodes`
	`475`	`+ else if ((keepComment \|\| child.type !== Comment) && child.type !== Text) {`
`474`	`476`	`ret.push(child)`
`475`	`477`	`}`
`476`	`478`	`}`