Skip to content
Permalink
Browse files

Limit the scope of the CSS 'auto' change to just height/width. Fixes …

…#7293.
  • Loading branch information...
jeresig committed Nov 3, 2010
1 parent 335e0a3 commit 3e38a861fee065fa1a6e7678eefe90b2379fed8e
Showing with 7 additions and 3 deletions.
  1. +2 −2 src/css.js
  2. +5 −1 test/unit/effects.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;
};
}

@@ -6,14 +6,18 @@ test("sanity check", function() {
});

test("show()", function() {
expect(27);
expect(28);

var hiddendiv = jQuery("div.hidden");

hiddendiv.hide().show();

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");

0 comments on commit 3e38a86

Please sign in to comment.
You can’t perform that action at this time.