feat: feat: add warning for ambiguous combined usage of slot-scope an…

…d v-for close vuejs#6817
lovelope · Feb 1, 2018 · 6b9a511 · 6b9a511
1 parent b6c3697
commit 6b9a511
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/src/compiler/parser/index.js b/src/compiler/parser/index.js
@@ -460,6 +460,15 @@ function processSlot (el) {
       }
       el.slotScope = slotScope || getAndRemoveAttr(el, 'slot-scope')
     } else if ((slotScope = getAndRemoveAttr(el, 'slot-scope'))) {
+      /* istanbul ignore if */
+      if (process.env.NODE_ENV !== 'production' && el.attrsMap['v-for']) {
+        warn(
+          `Ambiguous combined usage of slot-scope and v-for on <${el.tag}> ` +
+          `(v-for takes higher priority). Use a wrapper <template> for the ` +
+          `scoped slot to make it clearer.`,
+          true
+        )
+      }
       el.slotScope = slotScope
     }
     const slotTarget = getBindingAttr(el, 'slot')