Landing pull request 553. Fallback to elem.style for disconnected nodes in width...

More Details:
 - https://github.com/jquery/jquery/pull/553
 - http://bugs.jquery.com/ticket/10254
 - http://bugs.jquery.com/ticket/8388

diff --git a/src/css.js b/src/css.js
index 2adfdffe1ed4e2b40297f3390e78907a126a9cf7..ba645fb463005036b9baf25aa1026505bef23dc1 100644 (file)
--- a/src/css.js
+++ b/src/css.js
@@ -290,6 +290,10 @@ if ( document.documentElement.currentStyle ) {
                        rsLeft = elem.runtimeStyle && elem.runtimeStyle[ name ],
                        style = elem.style;
 
+               if ( ret === null && style ) {
+                       ret = style[ name ];
+               }
+
                // From the awesome hack by Dean Edwards
                // http://erik.eae.net/archives/2007/07/27/18.54.15/#comment-102291
 

diff --git a/test/unit/css.js b/test/unit/css.js
index 78c6def613c4e5503b67e72f6b780231ea7a4f15..7e9509593cde0ad223c1ecfe05c0e504593b9a0b 100644 (file)
--- a/test/unit/css.js
+++ b/test/unit/css.js
@@ -435,7 +435,7 @@ test(":visible selector works properly on children with a hidden parent (bug #45
 test("internal ref to elem.runtimeStyle (bug #7608)", function () {
        expect(1);
        var result = true;
-       
+
        try {
                jQuery("#foo").css( { width: "0%" } ).css("width");
        } catch (e) {
@@ -476,7 +476,7 @@ test("widows & orphans #8936", function () {
        var $p = jQuery("<p>").appendTo("#qunit-fixture");
 
        if ( "widows" in $p[0].style ) {
-               expect(4);      
+               expect(4);
                $p.css({
                        widows: 0,
                        orphans: 0
@@ -502,6 +502,15 @@ test("widows & orphans #8936", function () {
        $p.remove();
 });
 
+test("can't get css for disconnected in IE<9, see #10254 and #8388", function() {
+       expect( 2 );
+       var span = jQuery( "<span/>" ).css( "background-image", "url(http://static.jquery.com/files/rocker/images/logo_jquery_215x53.gif)" );
+       equal( span.css( "background-image" ), "url(http://static.jquery.com/files/rocker/images/logo_jquery_215x53.gif)", "can't get background-image in IE<9, see #10254" );
+
+       var div = jQuery( "<div/>" ).css( "top", 10 );
+       equal( div.css( "top" ), "10px", "can't get top in IE<9, see #8388" );
+});
+
 test("Do not append px to 'fill-opacity' #9548", 1, function() {
 
        var $div = jQuery("<div>").appendTo("#qunit-fixture");
@@ -510,4 +519,4 @@ test("Do not append px to 'fill-opacity' #9548", 1, function() {
                equal( jQuery(this).css("fill-opacity"), 1, "Do not append px to 'fill-opacity'");
        });
 
-});
+});
\ No newline at end of file