From cb828f3dab6ec8439ac111246346a8d4b3205fed Mon Sep 17 00:00:00 2001 From: Brandon Aaron Date: Wed, 25 Apr 2007 18:19:39 +0000 Subject: [PATCH] Fix for #1087: Using appendTo with select elements --- src/jquery/coreTest.js | 6 +++++- src/jquery/jquery.js | 6 +++--- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/src/jquery/coreTest.js b/src/jquery/coreTest.js index 049c4cc..05fd026 100644 --- a/src/jquery/coreTest.js +++ b/src/jquery/coreTest.js @@ -356,7 +356,7 @@ test("wrap(String|Element)", function() { }); test("append(String|Element|Array<Element>|jQuery)", function() { - expect(12); + expect(13); var defaultText = 'Try them out:' var result = $('#first').append('buga'); ok( result.text() == defaultText + 'buga', 'Check if text appending works' ); @@ -404,6 +404,10 @@ test("append(String|Element|Array<Element>|jQuery)", function() { ok( pass, "Test for appending a DOM node to the contents of an IFrame" ); + reset(); + $('#select1').appendTo('#foo'); + t( 'Append select', '#foo select', ['select1'] ); + }); test("appendTo(String|Element|Array<Element>|jQuery)", function() { diff --git a/src/jquery/jquery.js b/src/jquery/jquery.js index 5347c92..8859867 100644 --- a/src/jquery/jquery.js +++ b/src/jquery/jquery.js @@ -1525,10 +1525,10 @@ jQuery.extend({ arg = jQuery.makeArray( div.childNodes ); } - if ( arg.length === 0 && !jQuery.nodeName(arg, "form") ) + if ( arg.length === 0 && !jQuery(arg).is("form, select") ) return; - - if ( arg[0] == undefined || jQuery.nodeName(arg, "form") ) + + if ( arg[0] == undefined || jQuery(arg).is("form, select") ) r.push( arg ); else r = jQuery.merge( r, arg ); -- 1.7.10.4