Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Fix #12148. Let .toggle() call the public .hide() for punching.
There is a slightly shorter way to do this but it's not Closure-friendly.
  • Loading branch information
dmethvin committed Jul 26, 2012
1 parent 5119b25 commit ed898c6
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 8 deletions.
16 changes: 9 additions & 7 deletions src/css.js
Expand Up @@ -43,8 +43,7 @@ function vendorPropName( style, name ) {
return origName;
}

function isHidden( elem, el ) {
elem = el || elem;
function isHidden( elem ) {
return jQuery.css( elem, "display" ) === "none" || !jQuery.contains( elem.ownerDocument.documentElement, elem );

This comment has been minimized.

Copy link
@gibson042

gibson042 Jul 26, 2012

Member

Sizzle now supports contains( document, element ), so .documentElement here is unnecessary.

}

Expand Down Expand Up @@ -111,16 +110,19 @@ jQuery.fn.extend({
hide: function() {
return showHide( this );
},
toggle: function( fn, fn2 ) {
var bool = typeof fn === "boolean";
toggle: function( state, fn2 ) {
var bool = typeof state === "boolean";

if ( jQuery.isFunction( fn ) && jQuery.isFunction( fn2 ) ) {
if ( jQuery.isFunction( state ) && jQuery.isFunction( fn2 ) ) {
return eventsToggle.apply( this, arguments );
}

return this.each(function() {
var state = bool ? fn : jQuery( this ).is(":hidden");
showHide([ this ], state );
if ( bool ? state : isHidden( this ) ) {
jQuery( this ).show();
} else {
jQuery( this ).hide();
}
});
}
});
Expand Down
13 changes: 12 additions & 1 deletion test/unit/css.js
Expand Up @@ -561,7 +561,7 @@ test( "show() resolves correct default display, detached nodes (#10006)", functi
});

test("toggle()", function() {
expect(6);
expect(7);
var x = jQuery("#foo");
ok( x.is(":visible"), "is visible" );
x.toggle();
Expand All @@ -575,6 +575,17 @@ test("toggle()", function() {
ok( x.is(":hidden"), "is hidden" );
x.toggle(true);
ok( x.is(":visible"), "is visible again" );

// Ensure hide() is called when toggled (#12148)
var oldHide = jQuery.fn.hide;
jQuery.fn.hide = function() {
ok( true, name + " method called on toggle" );
return oldHide.apply( this, arguments );
};
x.toggle( name === "show" );
jQuery.fn.hide = oldHide;


});

test("hide hidden elements (bug #7141)", function() {
Expand Down

0 comments on commit ed898c6

Please sign in to comment.