]> source.dussan.org Git - jquery.git/commitdiff
Fixes #13088: under IE8, $(selector).attr('style') always return lowercase string
authorMike Sherov <mike.sherov@gmail.com>
Mon, 31 Dec 2012 17:30:02 +0000 (12:30 -0500)
committerMike Sherov <mike.sherov@gmail.com>
Mon, 31 Dec 2012 17:30:02 +0000 (12:30 -0500)
src/attributes.js
test/unit/attributes.js

index 6da68ef11be691072c50f099089757967cf01263..4193bbffb20572817c585cf7ecb30a66e634ae1b 100644 (file)
@@ -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 + "" );
index 84f406d1f116cee90a071c7724b5f696b2b5344c..072240049cdad4bf36a2d34e941737a674e63a91 100644 (file)
@@ -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("<div/>").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("<button>text</button>").insertAfter("#button");