Fix olddisplay was inappropriately set when calling hide on an already hidden element...
[jquery.git] / test / unit / effects.js
index 284eec3..2dfe834 100644 (file)
@@ -37,9 +37,11 @@ test("show()", function() {
                ok( pass, "Show with " + name + " does not call animate callback" );
        });
 
-       jQuery("#main").append('<div id="show-tests"><div><p><a href="#"></a></p><code></code><pre></pre><span></span></div><table><thead><tr><th></th></tr></thead><tbody><tr><td></td></tr></tbody></table><ul><li></li></ul></div>');
-
-       var old = jQuery("#show-tests table").show().css("display") !== "table";
+       // #show-tests * is set display: none in CSS
+       jQuery("#main").append('<div id="show-tests"><div><p><a href="#"></a></p><code></code><pre></pre><span></span></div><table><thead><tr><th></th></tr></thead><tbody><tr><td></td></tr></tbody></table><ul><li></li></ul></div><table id="test-table"></table>');
+       
+       var old = jQuery("#test-table").show().css("display") !== "table";
+       jQuery("#test-table").remove();
 
        var test = {
                "div"      : "block",
@@ -69,10 +71,12 @@ test("show(Number) - other displays", function() {
        QUnit.reset();
        stop();
 
-       jQuery("#main").append('<div id="show-tests"><div><p><a href="#"></a></p><code></code><pre></pre><span></span></div><table><thead><tr><th></th></tr></thead><tbody><tr><td></td></tr></tbody></table><ul><li></li></ul></div>');
+       // #show-tests * is set display: none in CSS
+       jQuery("#main").append('<div id="show-tests"><div><p><a href="#"></a></p><code></code><pre></pre><span></span></div><table><thead><tr><th></th></tr></thead><tbody><tr><td></td></tr></tbody></table><ul><li></li></ul></div><table id="test-table"></table>');
 
-       var old = jQuery("#show-tests table").show().css("display") !== "table",
+       var old = jQuery("#test-table").show().css("display") !== "table",
                num = 0;
+       jQuery("#test-table").remove();
 
        var test = {
                "div"      : "block",
@@ -92,10 +96,7 @@ test("show(Number) - other displays", function() {
        };
 
        jQuery.each(test, function(selector, expected) {
-               // IE sometimes has issues with chained functions referencing
-               // assignments from outside the closure
-               var elem = jQuery(selector, "#show-tests");
-               elem.show(1, function() {
+               var elem = jQuery(selector, "#show-tests").show(1, function() {
                        equals( elem.css("display"), expected, "Show using correct display type for " + selector );
                        if ( ++num === 15 ) {
                                start();
@@ -124,7 +125,7 @@ test("animate negative height", function() {
        });
 });
 
-test("animate inline width/height", function() {
+test("animate block as inline width/height", function() {
        expect(3);
        stop();
        jQuery("#foo").css({ display: "inline", width: '', height: '' }).animate({ width: 42, height: 42 }, 100, function() {
@@ -135,6 +136,20 @@ test("animate inline width/height", function() {
        });
 });
 
+test("animate native inline width/height", function() {
+       expect(3);
+       stop();
+       jQuery("#foo").css({ display: "", width: '', height: '' })
+               .append('<span>text</span>')
+               .children('span')
+                       .animate({ width: 42, height: 42 }, 100, function() {
+                               equals( jQuery(this).css("display"), "inline-block", "inline-block was set on non-floated inline element when animating width/height" );
+                               equals( this.offsetWidth, 42, "width was animated" );
+                               equals( this.offsetHeight, 42, "height was animated" );
+                               start();
+                       });
+});
+
 test("animate block width/height", function() {
        expect(3);
        stop();
@@ -158,12 +173,31 @@ test("animate table width/height", function() {
        });
 });
 
+test("animate table-row width/height", function() {
+       expect(3);
+       stop();
+       var tr = jQuery("#table")
+               .attr({ "cellspacing": 0, "cellpadding": 0, "border": 0 })
+               .html("<tr style='height:42px;'><td style='padding:0;'><div style='width:20px;height:20px;'></div></td></tr>")
+               .find("tr");
+
+       // IE<8 uses “block” instead of the correct display type
+       var displayMode = tr.css("display") !== "table-row" ? "block" : "table-row";
+
+       tr.animate({ width: 10, height: 10 }, 100, function() {
+               equals( jQuery(this).css("display"), displayMode, "display mode is correct" );
+               equals( this.offsetWidth, 20, "width animated to shrink wrap point" );
+               equals( this.offsetHeight, 20, "height animated to shrink wrap point" );
+               start();
+       });
+});
+
 test("animate table-cell width/height", function() {
        expect(3);
        stop();
        var td = jQuery("#table")
                .attr({ "cellspacing": 0, "cellpadding": 0, "border": 0 })
-               .append("<tr><td style='width:42px;height:42px;padding:0;'><div style='width:20px;height:20px;'></div></td></tr>")
+               .html("<tr><td style='width:42px;height:42px;padding:0;'><div style='width:20px;height:20px;'></div></td></tr>")
                .find("td");
 
        // IE<8 uses “block” instead of the correct display type
@@ -212,7 +246,7 @@ test("animate option (queue === false)", function () {
 
 test("animate with no properties", function() {
        expect(2);
-       
+
        var divs = jQuery("div"), count = 0;
 
        divs.animate({}, function(){
@@ -234,30 +268,30 @@ test("animate with no properties", function() {
 
 test("animate duration 0", function() {
        expect(11);
-       
+
        stop();
-       
+
        var $elems = jQuery([{ a:0 },{ a:0 }]), counter = 0;
-       
+
        equals( jQuery.timers.length, 0, "Make sure no animation was running from another test" );
-               
+
        $elems.eq(0).animate( {a:1}, 0, function(){
                ok( true, "Animate a simple property." );
                counter++;
        });
-       
+
        // Failed until [6115]
        equals( jQuery.timers.length, 0, "Make sure synchronic animations are not left on jQuery.timers" );
-       
+
        equals( counter, 1, "One synchronic animations" );
-       
+
        $elems.animate( { a:2 }, 0, function(){
                ok( true, "Animate a second simple property." );
                counter++;
        });
-       
+
        equals( counter, 3, "Multiple synchronic animations" );
-       
+
        $elems.eq(0).animate( {a:3}, 0, function(){
                ok( true, "Animate a third simple property." );
                counter++;
@@ -268,12 +302,12 @@ test("animate duration 0", function() {
                equals( counter, 5, "One synchronic and one asynchronic" );
                start();
        });
-       
+
        var $elem = jQuery("<div />");
-       $elem.show(0, function(){ 
+       $elem.show(0, function(){
                ok(true, "Show callback with no duration");
        });
-       $elem.hide(0, function(){ 
+       $elem.hide(0, function(){
                ok(true, "Hide callback with no duration");
        });
 });
@@ -415,7 +449,7 @@ test("toggle()", function() {
        ok( x.is(":hidden"), "is hidden" );
        x.toggle();
        ok( x.is(":visible"), "is visible again" );
-       
+
        x.toggle(true);
        ok( x.is(":visible"), "is visible" );
        x.toggle(false);
@@ -445,7 +479,7 @@ test("JS Overflow and Display", function() {
                .after("text after")
                .animate({ opacity: 0.5 }, "slow", jQuery.checkOverflowDisplay);
 });
-               
+
 test("CSS Overflow and Display", function() {
        expect(2);
        stop();
@@ -517,16 +551,16 @@ jQuery.each( {
        }, function(tn, t){
                test(fn + " to " + tn, function() {
                        var elem = jQuery.makeTest( fn + " to " + tn );
-       
+
                        var t_w = t( elem, "width" );
                        var f_w = f( elem, "width" );
                        var t_h = t( elem, "height" );
                        var f_h = f( elem, "height" );
                        var t_o = t( elem, "opacity" );
                        var f_o = f( elem, "opacity" );
-                       
+
                        var num = 0;
-                       
+
                        if ( t_h == "show" ) num++;
                        if ( t_w == "show" ) num++;
                        if ( t_w == "hide"||t_w == "show" ) num++;
@@ -536,52 +570,52 @@ jQuery.each( {
                        if ( t_o.constructor == Number ) num += 2;
                        if ( t_w.constructor == Number ) num += 2;
                        if ( t_h.constructor == Number ) num +=2;
-                       
+
                        expect(num);
                        stop();
-       
+
                        var anim = { width: t_w, height: t_h, opacity: t_o };
-       
+
                        elem.animate(anim, 50, function(){
                                if ( t_w == "show" )
                                        equals( this.style.display, "block", "Showing, display should block: " + this.style.display);
-                                       
+
                                if ( t_w == "hide"||t_w == "show" )
                                        ok(f_w === "" ? this.style.width === f_w : this.style.width.indexOf(f_w) === 0, "Width must be reset to " + f_w + ": " + this.style.width);
-                                       
+
                                if ( t_h == "hide"||t_h == "show" )
                                        ok(f_h === "" ? this.style.height === f_h : this.style.height.indexOf(f_h) === 0, "Height must be reset to " + f_h + ": " + this.style.height);
-                                       
+
                                var cur_o = jQuery.style(this, "opacity");
 
                                if ( t_o == "hide" || t_o == "show" )
                                        equals(cur_o, f_o, "Opacity must be reset to " + f_o + ": " + cur_o);
-                                       
+
                                if ( t_w == "hide" )
                                        equals(this.style.display, "none", "Hiding, display should be none: " + this.style.display);
-                                       
+
                                if ( t_o.constructor == Number ) {
                                        equals(cur_o, t_o, "Final opacity should be " + t_o + ": " + cur_o);
-                                       
+
                                        ok(jQuery.css(this, "opacity") != "" || cur_o == t_o, "Opacity should be explicitly set to " + t_o + ", is instead: " + cur_o);
                                }
-                                       
+
                                if ( t_w.constructor == Number ) {
                                        equals(this.style.width, t_w + "px", "Final width should be " + t_w + ": " + this.style.width);
-                                       
+
                                        var cur_w = jQuery.css(this,"width");
 
                                        ok(this.style.width != "" || cur_w == t_w, "Width should be explicitly set to " + t_w + ", is instead: " + cur_w);
                                }
-                                       
+
                                if ( t_h.constructor == Number ) {
                                        equals(this.style.height, t_h + "px", "Final height should be " + t_h + ": " + this.style.height);
-                                       
+
                                        var cur_h = jQuery.css(this,"height");
 
                                        ok(this.style.height != "" || cur_h == t_h, "Height should be explicitly set to " + t_h + ", is instead: " + cur_w);
                                }
-                               
+
                                if ( t_h == "show" ) {
                                        var old_h = jQuery.css(this, "height");
                                        jQuery(this).append("<br/>Some more text<br/>and some more...");
@@ -592,23 +626,23 @@ jQuery.each( {
                                                equals(jQuery.css(this, "height"), old_h, "Make sure height is not auto.");
                                        }
                                }
-       
+
                                start();
                        });
                });
        });
 });
 
-jQuery.fn.saveState = function(){
-       var check = ['opacity','height','width','display','overflow'];  
+jQuery.fn.saveState = function(hiddenOverflow){
+       var check = ['opacity','height','width','display','overflow'];
        expect(check.length);
-       
+
        stop();
        return this.each(function(){
                var self = this;
                self.save = {};
                jQuery.each(check, function(i,c){
-                       self.save[c] = self.style[ c ] || jQuery.css(self,c);
+                       self.save[c] = c === "overflow" && hiddenOverflow ? "hidden" : self.style[ c ] || jQuery.css(self,c);
                });
        });
 };
@@ -631,39 +665,39 @@ test("Chain fadeIn fadeOut", function() {
 });
 
 test("Chain hide show", function() {
-       jQuery('#show div').saveState().hide('fast').show('fast',jQuery.checkState);
+       jQuery('#show div').saveState(jQuery.support.shrinkWrapBlocks).hide('fast').show('fast',jQuery.checkState);
 });
 test("Chain show hide", function() {
-       jQuery('#hide div').saveState().show('fast').hide('fast',jQuery.checkState);
+       jQuery('#hide div').saveState(jQuery.support.shrinkWrapBlocks).show('fast').hide('fast',jQuery.checkState);
 });
 test("Chain show hide with easing and callback", function() {
        jQuery('#hide div').saveState().show('fast').hide('fast','linear',jQuery.checkState);
 });
 
 test("Chain toggle in", function() {
-       jQuery('#togglein div').saveState().toggle('fast').toggle('fast',jQuery.checkState);
+       jQuery('#togglein div').saveState(jQuery.support.shrinkWrapBlocks).toggle('fast').toggle('fast',jQuery.checkState);
 });
 test("Chain toggle out", function() {
-       jQuery('#toggleout div').saveState().toggle('fast').toggle('fast',jQuery.checkState);
+       jQuery('#toggleout div').saveState(jQuery.support.shrinkWrapBlocks).toggle('fast').toggle('fast',jQuery.checkState);
 });
 test("Chain toggle out with easing and callback", function() {
- jQuery('#toggleout div').saveState().toggle('fast').toggle('fast','linear',jQuery.checkState);
+ jQuery('#toggleout div').saveState(jQuery.support.shrinkWrapBlocks).toggle('fast').toggle('fast','linear',jQuery.checkState);
 });
 test("Chain slideDown slideUp", function() {
-       jQuery('#slidedown div').saveState().slideDown('fast').slideUp('fast',jQuery.checkState);
+       jQuery('#slidedown div').saveState(jQuery.support.shrinkWrapBlocks).slideDown('fast').slideUp('fast',jQuery.checkState);
 });
 test("Chain slideUp slideDown", function() {
-       jQuery('#slideup div').saveState().slideUp('fast').slideDown('fast',jQuery.checkState);
+       jQuery('#slideup div').saveState(jQuery.support.shrinkWrapBlocks).slideUp('fast').slideDown('fast',jQuery.checkState);
 });
 test("Chain slideUp slideDown with easing and callback", function() {
-       jQuery('#slideup div').saveState().slideUp('fast').slideDown('fast','linear',jQuery.checkState);
+       jQuery('#slideup div').saveState(jQuery.support.shrinkWrapBlocks).slideUp('fast').slideDown('fast','linear',jQuery.checkState);
 });
 
 test("Chain slideToggle in", function() {
-       jQuery('#slidetogglein div').saveState().slideToggle('fast').slideToggle('fast',jQuery.checkState);
+       jQuery('#slidetogglein div').saveState(jQuery.support.shrinkWrapBlocks).slideToggle('fast').slideToggle('fast',jQuery.checkState);
 });
 test("Chain slideToggle out", function() {
-       jQuery('#slidetoggleout div').saveState().slideToggle('fast').slideToggle('fast',jQuery.checkState);
+       jQuery('#slidetoggleout div').saveState(jQuery.support.shrinkWrapBlocks).slideToggle('fast').slideToggle('fast',jQuery.checkState);
 });
 
 test("Chain fadeTo 0.5 1.0 with easing and callback)", function() {
@@ -704,26 +738,26 @@ test("jQuery.show('fast') doesn't clear radio buttons (bug #1095)", function ()
 });
 
 test("animate with per-property easing", function(){
-       
+
        expect(3);
        stop();
-       
+
        var _test1_called = false;
        var _test2_called = false;
        var _default_test_called = false;
-       
+
        jQuery.easing['_test1'] = function() {
                _test1_called = true;
        };
-       
+
        jQuery.easing['_test2'] = function() {
                _test2_called = true;
        };
-       
+
        jQuery.easing['_default_test'] = function() {
                _default_test_called = true;
        };
-       
+
        jQuery({a:0,b:0,c:0}).animate({
                a: [100, '_test1'],
                b: [100, '_test2'],
@@ -734,5 +768,35 @@ test("animate with per-property easing", function(){
                ok(_test2_called, "Easing function (2) called");
                ok(_default_test_called, "Easing function (_default) called");
        });
+
+});
+
+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();
+               });
+       });
 });