Merge pull request madrobby#1095 from caricati/master

Better support for array argument to `append`

src/zepto.js

@@ -849,8 +849,17 @@ var Zepto = (function() {
     $.fn[operator] = function(){
       // arguments can be nodes, arrays of nodes, Zepto objects and HTML strings
       var argType, nodes = $.map(arguments, function(arg) {
+            var arr = []
             argType = type(arg)
-            return argType == "object" || argType == "array" || arg == null ?
+            if (argType == "array") {
+              arg.forEach(function(el) {
+                if (el.nodeType !== undefined) return arr.push(el)
+                else if ($.zepto.isZ(el)) return arr = arr.concat(el.get())
+                arr = arr.concat(zepto.fragment(el))
+              })
+              return arr
+            }
+            return argType == "object" || arg == null ?
               arg : zepto.fragment(arg)
           }),
           parent, copyByClone = this.length > 1

test/zepto.html

@@ -2186,6 +2186,19 @@ <h1>Zepto Core unit tests</h1>
           $.map(el.children(), function(c){ return $(c).text() }).join(' '))
       },
 
+      testAppendWithArrayParams: function(t){
+        var zpElement = $('<b>five</b>')
+        var multElements = $('<b>six</b><b>seven</b>')
+        var node = document.createElement('b')
+        node.appendChild(document.createTextNode('four'))
+
+        // append elements on this array
+        var arrElements = ['<b>one</b><b>two</b>', '<b>three</b>', node, zpElement, multElements ]
+        var el = $('<div></div>')
+        el.append(arrElements)
+        t.assertEqual('<b>one</b><b>two</b><b>three</b><b>four</b><b>five</b><b>six</b><b>seven</b>', el.html())
+      },
+
       testAppendToPrependTo: function(t){
         // testing with Zepto object
         function div(contents){