improve fragment instance attr warning check (close vuejs#1669)

yyx990803 · yyx990803 · commit 7da156e71c38 · 2015-11-02T12:30:58.000-05:00
diff --git a/src/compiler/compile.js b/src/compiler/compile.js
@@ -213,15 +213,20 @@ exports.compileRoot = function (el, options, contextOptions) {
     // warn container directives for fragment instances
     var names = containerAttrs
       .filter(function (attr) {
-        return attr.name.indexOf('_v-') < 0
+        // allow vue-loader/vueify scoped css attributes
+        return attr.name.indexOf('_v-') < 0 &&
+          // allow event listeners
+          !onRE.test(attr.name) &&
+          // allow slots
+          attr.name !== 'slot'
       })
       .map(function (attr) {
         return '"' + attr.name + '"'
       })
     if (names.length) {
       var plural = names.length > 1
       _.warn(
-        'Attribute' + (plural ? 's ' : ' ') + names.join(',') +
+        'Attribute' + (plural ? 's ' : ' ') + names.join(', ') +
         (plural ? ' are' : ' is') + ' ignored on component ' +
         '<' + options.el.tagName.toLowerCase() + '> because ' +
         'the component is a fragment instance: ' +
