From 9d1bfeb7ffe63b5a7488929b7be847bc1a00f6a6 Mon Sep 17 00:00:00 2001 From: jeresig Date: Mon, 1 Nov 2010 19:04:59 -0400 Subject: [PATCH] Make sure that accessing computed CSS for elements returns 'auto' instead of '' consistently. Fixes #7337. --- src/css.js | 9 +++++---- test/unit/css.js | 4 ++-- 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/src/css.js b/src/css.js index 30cecf3..0998657 100644 --- a/src/css.js +++ b/src/css.js @@ -173,12 +173,13 @@ jQuery.each(["height", "width"], function( i, name ) { val = curCSS( elem, name, name ); if ( val != null ) { - return val === "auto" ? "" : val; + return val; } } if ( val < 0 || val == null ) { - return elem.style[ name ]; + val = elem.style[ name ]; + return val === "" ? "auto" : val; } return typeof val === "string" ? val : val + "px"; @@ -247,7 +248,7 @@ if ( getComputedStyle ) { } } - return ret; + return ret === "" ? "auto" : ret; }; } else if ( document.documentElement.currentStyle ) { @@ -274,7 +275,7 @@ if ( getComputedStyle ) { elem.runtimeStyle.left = rsLeft; } - return ret; + return ret === "" ? "auto" : ret; }; } diff --git a/test/unit/css.js b/test/unit/css.js index 4ec7c60..71f8835 100644 --- a/test/unit/css.js +++ b/test/unit/css.js @@ -13,8 +13,8 @@ test("css(String|Hash)", function() { var div = jQuery( "
" ); - equals( div.css("width"), "", "Width on disconnected node." ); - equals( div.css("height"), "", "Height on disconnected node." ); + equals( div.css("width"), "auto", "Width on disconnected node." ); + equals( div.css("height"), "auto", "Height on disconnected node." ); div.css({ width: 4, height: 4 }); -- 1.7.10.4