]> source.dussan.org Git - jquery.git/commitdiff
Make sure that events are cloned for wrap, fixes #2977.
authorJohn Resig <jeresig@gmail.com>
Tue, 8 Dec 2009 19:21:24 +0000 (11:21 -0800)
committerJohn Resig <jeresig@gmail.com>
Tue, 8 Dec 2009 19:21:24 +0000 (11:21 -0800)
src/manipulation.js
test/unit/manipulation.js

index 1fe82fa27ebe86b9addc06cfbc36d72ad0fe8a0a..0c2753e968200c98c7b983ebfaf58f649740ded9 100644 (file)
@@ -48,7 +48,7 @@ jQuery.fn.extend({
 
                if ( this[0] ) {
                        // The elements to wrap the target around
-                       var wrap = jQuery( html, this[0].ownerDocument ).eq(0).clone();
+                       var wrap = jQuery( html, this[0].ownerDocument ).eq(0).clone(true);
 
                        if ( this[0].parentNode ) {
                                wrap.insertBefore( this[0] );
index 44fafa92b1cb09eb3838b18dcf70989145abe546..4631ead61eac05a8142aec381f4b0e51a79da61d 100644 (file)
@@ -13,7 +13,7 @@ test("text()", function() {
 });
 
 var testWrap = function(val) {
-       expect(15);
+       expect(18);
        var defaultText = 'Try them out:'
        var result = jQuery('#first').wrap(val( '<div class="red"><span></span></div>' )).text();
        equals( defaultText, result, 'Check for wrapping of on-the-fly html' );
@@ -54,6 +54,20 @@ var testWrap = function(val) {
        equals( j[0].parentNode.parentNode.childNodes.length, 1, "There should only be one element wrapping." );
        equals( j.length, 1, "There should only be one element (no cloning)." );
        equals( j[0].parentNode.nodeName.toUpperCase(), "P", "The span should be in the paragraph." );
+
+       // Wrap an element with a jQuery set
+       j = jQuery("<span/>").wrap(jQuery("<div></div>"));
+       equals( j[0].parentNode.nodeName.toLowerCase(), "div", "Wrapping works." );
+
+       // Wrap an element with a jQuery set and event
+       result = jQuery("<div></div>").click(function(){
+               ok(true, "Event triggered.");
+       });
+
+       j = jQuery("<span/>").wrap(result);
+       equals( j[0].parentNode.nodeName.toLowerCase(), "div", "Wrapping works." );
+
+       j.parent().trigger("click");
 }
 
 test("wrap(String|Element)", function() {