Fixes #7397; 4 supporting unit tests
This commit is contained in:
parent
a64dc04050
commit
118c8c4600
9
src/effects.js
vendored
9
src/effects.js
vendored
|
@ -60,11 +60,12 @@ jQuery.fn.extend({
|
|||
|
||||
} else {
|
||||
for ( var i = 0, j = this.length; i < j; i++ ) {
|
||||
var display = jQuery.css( this[i], "display" );
|
||||
var display = jQuery.css( this[i], "display" ),
|
||||
old = jQuery.data( this[i], "olddisplay" );
|
||||
|
||||
if ( display !== "none" ) {
|
||||
jQuery.data( this[i], "olddisplay", display );
|
||||
}
|
||||
if ( !old && display !== "none" ) {
|
||||
jQuery.data( this[i], "olddisplay", display );
|
||||
}
|
||||
}
|
||||
|
||||
// Set the display of the elements in a second loop
|
||||
|
|
35
test/unit/effects.js
vendored
35
test/unit/effects.js
vendored
|
@ -130,6 +130,41 @@ test("show(Number) - other displays", function() {
|
|||
});
|
||||
});
|
||||
|
||||
|
||||
|
||||
// Supports #7397
|
||||
test("Persist correct display value", function() {
|
||||
expect(4);
|
||||
QUnit.reset();
|
||||
stop();
|
||||
|
||||
// #show-tests * is set display: none in CSS
|
||||
jQuery("#main").append('<div id="show-tests"><span style="position:absolute;"></span></div>');
|
||||
|
||||
var $span = jQuery("#show-tests span"),
|
||||
orig = $span.css("display"),
|
||||
num = 0;
|
||||
|
||||
equal(orig, "none", "Expecting to start at display: none");
|
||||
|
||||
$span.text('Saving...').fadeIn(100, function() {
|
||||
|
||||
equal($span.css("display"), "block", "Expecting display: block");
|
||||
|
||||
$span.text('Saved!').fadeOut(100, function () {
|
||||
|
||||
equal($span.css("display"), "none", "Expecting display: none");
|
||||
|
||||
$span.text('Saving...').fadeIn(100, function() {
|
||||
|
||||
equal($span.css("display"), "block", "Expecting display: block");
|
||||
|
||||
start();
|
||||
});
|
||||
});
|
||||
});
|
||||
});
|
||||
|
||||
test("animate(Hash, Object, Function)", function() {
|
||||
expect(1);
|
||||
stop();
|
||||
|
|
Loading…
Reference in a new issue