From abead1c86b75d376544e52cd916031a67caf2c34 Mon Sep 17 00:00:00 2001 From: Mike Sherov Date: Mon, 31 Dec 2012 12:30:02 -0500 Subject: [PATCH] Fixes #13088: under IE8, $(selector).attr('style') always return lowercase string --- src/attributes.js | 5 +++-- test/unit/attributes.js | 7 +++++-- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/src/attributes.js b/src/attributes.js index 6da68ef11..4193bbffb 100644 --- a/src/attributes.js +++ b/src/attributes.js @@ -600,8 +600,9 @@ if ( !jQuery.support.style ) { jQuery.attrHooks.style = { get: function( elem ) { // Return undefined in the case of empty string - // Normalize to lowercase since IE uppercases css property names - return elem.style.cssText.toLowerCase() || undefined; + // Note: IE uppercases css property names, but if we were to .toLowerCase() + // .cssText, that would destroy case senstitivity in URL's, like in "background" + return elem.style.cssText || undefined; }, set: function( elem, value ) { return ( elem.style.cssText = value + "" ); diff --git a/test/unit/attributes.js b/test/unit/attributes.js index 84f406d1f..072240049 100644 --- a/test/unit/attributes.js +++ b/test/unit/attributes.js @@ -130,8 +130,11 @@ test( "attr(String)", function() { equal( $img.attr("height"), "53", "Retrieve height attribute an an element with display:none." ); // Check for style support - ok( !!~jQuery("#dl").attr("style").indexOf("position"), "Check style attribute getter, also normalize css props to lowercase" ); - ok( !!~jQuery("#foo").attr("style", "position:absolute;").attr("style").indexOf("position"), "Check style setter" ); + var styleElem = jQuery("
").appendTo("#qunit-fixture").css({ + background: "url(UPPERlower.gif)" + }); + ok( !!~styleElem.attr("style").indexOf("UPPERlower.gif"), "Check style attribute getter" ); + ok( !!~styleElem.attr("style", "position:absolute;").attr("style").indexOf("absolute"), "Check style setter" ); // Check value on button element (#1954) var $button = jQuery("").insertAfter("#button"); -- 2.39.5