Fixed false negatives for whitespace in `vue/require-toggle-inside-tr…

…ansition` rule (#2293)
vuejs · Oct 24, 2023 · 467631e · 467631e
1 parent 15b99af
commit 467631e
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/lib/rules/require-toggle-inside-transition.js b/lib/rules/require-toggle-inside-transition.js
@@ -47,7 +47,8 @@ module.exports = {
     return utils.defineTemplateBodyVisitor(context, {
       /** @param {VElement} node */
       "VElement[name='transition'] > VElement"(node) {
-        if (node.parent.children[0] !== node) {
+        const child = node.parent.children.find(utils.isVElement)
+        if (child !== node) {
           return
         }
         verifyInsideElement(node)

diff --git a/tests/lib/rules/require-toggle-inside-transition.js b/tests/lib/rules/require-toggle-inside-transition.js
@@ -97,6 +97,11 @@ tester.run('require-toggle-inside-transition', rule, {
       filename: 'test.vue',
       code: '<template><transition><template v-for="e in list"><div /></template></transition></template>',
       errors: [{ messageId: 'expected' }]
+    },
+    {
+      filename: 'test.vue',
+      code: '<template><Transition>  <div /></Transition></template>',
+      errors: [{ messageId: 'expected' }]
     }
   ]
 })