deac/jquery
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Fix olddisplay was inappropriately set when calling hide on an already hidden element. Fixes #7141.

Browse source
This commit is contained in:
Colin Snover 2010-10-11 15:22:43 -05:00
parent 2866f16c09
commit 5646a4feee
2 changed files with 34 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

30
test/unit/effects.js vendored
View file

@ -770,3 +770,33 @@ test("animate with per-property easing", function(){
});
});
test("hide hidden elements (bug #7141)", function() {
expect(3);
QUnit.reset();
var div = jQuery("<div style='display:none'></div>").appendTo("#main");
equals( div.css("display"), "none", "Element is hidden by default" );
div.hide();
ok( !div.data("olddisplay"), "olddisplay is undefined after hiding an already-hidden element" );
div.show();
equals( div.css("display"), "block", "Show a double-hidden element" );
div.remove();
});
test("hide hidden elements, with animation (bug #7141)", function() {
expect(3);
QUnit.reset();
stop();
var div = jQuery("<div style='display:none'></div>").appendTo("#main");
equals( div.css("display"), "none", "Element is hidden by default" );
div.hide(1, function () {
ok( !div.data("olddisplay"), "olddisplay is undefined after hiding an already-hidden element" );
div.show(1, function () {
equals( div.css("display"), "block", "Show a double-hidden element" );
start();
});
});
});