aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorjeresig <jeresig@gmail.com>2010-11-03 18:59:55 -0400
committerjeresig <jeresig@gmail.com>2010-11-03 18:59:55 -0400
commitd9478954b9de79a7f3538e935bfd98cdc61ee435 (patch)
treeec5f7f77493ed170661ca50b8726b6bf2c5132b2
parent335e0a33e19b9c8703246dd033dcc68c2da5de36 (diff)
downloadjquery-d9478954b9de79a7f3538e935bfd98cdc61ee435.tar.gz
jquery-d9478954b9de79a7f3538e935bfd98cdc61ee435.zip
Limit the scope of the CSS 'auto' change to just height/width. Fixes #7393.
-rw-r--r--src/css.js4
-rw-r--r--test/unit/effects.js6
2 files changed, 7 insertions, 3 deletions
diff --git a/src/css.js b/src/css.js
index 0f1491466..1fbee3f47 100644
--- a/src/css.js
+++ b/src/css.js
@@ -177,7 +177,7 @@ jQuery.each(["height", "width"], function( i, name ) {
}
if ( val != null ) {
- return val;
+ return val === "" ? "auto" : val;
}
}
@@ -252,7 +252,7 @@ if ( document.defaultView && document.defaultView.getComputedStyle ) {
}
}
- return ret === "" ? "auto" : ret;
+ return ret;
};
}
diff --git a/test/unit/effects.js b/test/unit/effects.js
index 8facb4f54..74b336f1f 100644
--- a/test/unit/effects.js
+++ b/test/unit/effects.js
@@ -6,7 +6,7 @@ test("sanity check", function() {
});
test("show()", function() {
- expect(27);
+ expect(28);
var hiddendiv = jQuery("div.hidden");
@@ -14,6 +14,10 @@ test("show()", function() {
equals( hiddendiv.css("display"), "block", "Make sure a pre-hidden div is visible." );
+ var div = jQuery("<div>").hide().appendTo("body").show();
+
+ equal( div.css("display"), "block", "Make sure pre-hidden divs show" );
+
QUnit.reset();
hiddendiv = jQuery("div.hidden");