github/jquery
1
1
Fork 0
mirror of https://github.com/jquery/jquery.git synced 2026-02-06 17:24:54 -05:00
Code Issues Packages Projects Releases Wiki Activity

Fix #12148. Let .toggle() call the public .hide() for punching.

Browse Source 
There is a slightly shorter way to do this but it's not Closure-friendly.
This commit is contained in:
Dave Methvin
2012-07-25 21:24:49 -04:00
parent 5119b252ac
commit ed898c62c8
2 changed files with 21 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
src/css.js
View File

@@ -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 );
}
@@ -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();
}
});
}
});