Ticket #2184: 2184.diff

test/unit/core.js

@@ -955 +955 @@
 });
 
 test("clone()", function() {
-        expect(4);
+        expect(20);
         ok( 'This is a normal link: Yahoo' == $('#en').text(), 'Assert text for #en' );
         var clone = $('#yahoo').clone();
         ok( 'Try them out:Yahoo' == $('#first').append(clone).text(), 'Check for clone' );
         ok( 'This is a normal link: Yahoo' == $('#en').text(), 'Reassert text for #en' );
+
+        var cloneTags = [ 
+                "<table/>", "<tr/>", "<td/>", "<div/>", 
+                "<button/>", "<ul/>", "<ol/>", "<li/>",
+                "<input type='checkbox' />", "<select/>", "<option/>", "<textarea/>",
+                "<tbody/>", "<thead/>", "<tfoot/>", "<iframe/>"
+        ];
+        for (var i = 0; i < cloneTags.length; i++) {
+                var j = $(cloneTags[i]);
+                equals( j[0].tagName, j.clone()[0].tagName, 'Clone a &lt;' + cloneTags[i].substring(1));
+        }
+
         // using contents will get comments regular, text, and comment nodes
         var cl = $("#nonnodes").contents().clone();
         ok( cl.length >= 2, "Check node,textnode,comment clone works (some browsers delete comments on clone)" );

src/core.js

@@ -303 +303 @@
                                 // as properties will not be copied (such as the
                                 // the name attribute on an input).
                                 var clone = this.cloneNode(true),
-                                        container = document.createElement("div"),
-                                        container2 = document.createElement("div");
+                                        container = document.createElement("div");
                                 container.appendChild(clone);
-                                container2.innerHTML = container.innerHTML;
-                                return container2.firstChild;
+                                return jQuery.clean([container.innerHTML])[0];
                         } else
                                 return this.cloneNode(true);
                 });