From: Brandon Aaron Date: Wed, 25 Apr 2007 19:48:00 +0000 (+0000) Subject: Fix for #1086: Append all caps HTML tags X-Git-Url: http://git.asbjorn.biz/?p=jquery.git;a=commitdiff_plain;h=5fae496933cf4cad5826fbd871d0db455dc2947e Fix for #1086: Append all caps HTML tags --- diff --git a/src/jquery/coreTest.js b/src/jquery/coreTest.js index 9791767..0592ae9 100644 --- a/src/jquery/coreTest.js +++ b/src/jquery/coreTest.js @@ -405,17 +405,17 @@ 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'] ); - - reset(); $('
').appendTo('#form').append('test'); t( 'Append legend', '#legend', ['legend'] ); + reset(); + $('#select1').append(''); + ok( $('#select1 option:last').text() == "Test", "Appending <OPTION> (all caps)" ); + }); test("appendTo(String|Element|Array<Element>|jQuery)", function() { - expect(5); + expect(6); var defaultText = 'Try them out:' $('buga').appendTo('#first'); ok( $("#first").text() == defaultText + 'buga', 'Check if text appending works' ); @@ -435,6 +435,10 @@ test("appendTo(String|Element|Array<Element>|jQuery)", function() { expected = "This link has class=\"blog\": Simon Willison's WeblogTry them out:Yahoo"; $("#first, #yahoo").appendTo('#sap'); ok( expected == $('#sap').text(), "Check for appending of jQuery object" ); + + reset(); + $('#select1').appendTo('#foo'); + t( 'Append select', '#foo select', ['select1'] ); }); test("prepend(String|Element|Array<Element>|jQuery)", function() { diff --git a/src/jquery/jquery.js b/src/jquery/jquery.js index a2a9b9f..7b72185 100644 --- a/src/jquery/jquery.js +++ b/src/jquery/jquery.js @@ -1479,7 +1479,7 @@ jQuery.extend({ // Convert html string into DOM nodes if ( typeof arg == "string" ) { // Trim whitespace, otherwise indexOf won't work as expected - var s = jQuery.trim(arg), div = doc.createElement("div"), tb = []; + var s = jQuery.trim(arg).toLowerCase(), div = doc.createElement("div"), tb = []; var wrap = // option or optgroup