Skip to content
Permalink
Browse files

Fixes #7397; Adds 4 supporting unit tests.

  • Loading branch information...
rwaldron authored and jeresig committed Nov 10, 2010
1 parent a64dc04 commit 795e880bba1f7f949df58748f7fd92e50296a8f4
Showing with 39 additions and 4 deletions.
  1. +4 −4 src/effects.js
  2. +35 −0 test/unit/effects.js
@@ -61,10 +61,10 @@ jQuery.fn.extend({
} else {
for ( var i = 0, j = this.length; i < j; i++ ) {
var display = jQuery.css( this[i], "display" );

if ( display !== "none" ) {
jQuery.data( this[i], "olddisplay", display );
}
if ( !jQuery.data( this[i], "olddisplay" ) && display !== "none" ) {
jQuery.data( this[i], "olddisplay", display );
}
}

// Set the display of the elements in a second loop
@@ -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();

0 comments on commit 795e880

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