From 3842246024475eafdc8a00a7ac4358d06d76cab2 Mon Sep 17 00:00:00 2001 From: Dave Methvin Date: Fri, 9 Oct 2015 15:52:29 -0400 Subject: CSS: Make show/hide/toggle methods a module Unit test changes some uses of .show() and .hide() to .css( "display", ... ), there was already an implicit assumption in several of the existing tests. Fixes gh-2193 Close gh-2648 (cherry picked from commit 67d7a2eefee768b59eb3d51cb1fb2c671873e58a) Conflicts: Gruntfile.js src/css.js src/css/showHide.js test/unit/css.js --- src/css/showHide.js | 26 ++++++++++++++++++++++++-- 1 file changed, 24 insertions(+), 2 deletions(-) (limited to 'src/css') diff --git a/src/css/showHide.js b/src/css/showHide.js index efc1bdd3f..d143d4885 100644 --- a/src/css/showHide.js +++ b/src/css/showHide.js @@ -1,4 +1,6 @@ -define( [], function() { +define( [ + "../css/var/isHidden" +], function( isHidden ) { function showHide( elements, show ) { var display, elem, @@ -41,6 +43,26 @@ function showHide( elements, show ) { return elements; } -return showHide; +jQuery.fn.extend( { + show: function() { + return showHide( this, true ); + }, + hide: function() { + return showHide( this ); + }, + toggle: function( state ) { + if ( typeof state === "boolean" ) { + return state ? this.show() : this.hide(); + } + + return this.each( function() { + if ( isHidden( this ) ) { + jQuery( this ).show(); + } else { + jQuery( this ).hide(); + } + } ); + } +} ); } ); -- cgit v1.2.3