diff --git a/src/effects.js b/src/effects.js index c843549c2..f6b6dbd77 100644 --- a/src/effects.js +++ b/src/effects.js @@ -160,11 +160,8 @@ function defaultPrefilter( elem, props, opts ) { // Set display property to inline-block for height/width // animations on inline elements that are having width/height animated display = jQuery.css( elem, "display" ); - dDisplay = defaultDisplay( elem.nodeName ); - if ( display === "none" ) { - display = dDisplay; - } - if ( display === "inline" && + // Test default display if display is currently "none" + if ( (display === "none" ? defaultDisplay( elem.nodeName ) : display) === "inline" && jQuery.css( elem, "float" ) === "none" ) { // inline-level elements accept inline-block; @@ -204,6 +201,10 @@ function defaultPrefilter( elem, props, opts ) { } } orig[ prop ] = dataShow && dataShow[ prop ] || jQuery.style( elem, prop ); + + // Any non-fx value stops us from restoring the original display value + } else { + display = undefined; } } @@ -245,6 +246,10 @@ function defaultPrefilter( elem, props, opts ) { } } } + + // If this is a noop like .hide().hide(), restore an overwritten display value + } else if ( (display === "none" ? defaultDisplay( elem.nodeName ) : display) === "inline" ) { + style.display = display; } } diff --git a/test/unit/effects.js b/test/unit/effects.js index ca0754083..a27851459 100644 --- a/test/unit/effects.js +++ b/test/unit/effects.js @@ -1672,6 +1672,19 @@ test( "hide, fadeOut and slideUp called on element width height and width = 0 sh this.clock.tick( 400 ); }); +test( "hide should not leave hidden inline elements visible (#14848)", 2, function() { + var el = jQuery("#simon1"); + + el.hide( 1, function() { + equal( el.css( "display" ), "none", "hidden" ); + el.hide( 1, function() { + equal( el.css( "display" ), "none", "still hidden" ); + }); + }); + + this.clock.tick( 100 ); +}); + test( "Handle queue:false promises", 10, function() { var foo = jQuery( "#foo" ).clone().andSelf(), step = 1;