From 36774226dd241ae104e10e8073a0586c50ae59ad Mon Sep 17 00:00:00 2001 From: John Resig Date: Sun, 18 Mar 2007 18:52:09 +0000 Subject: [PATCH] Only fadeIn/Out or slideUp/Down elements that are already hidden/visible (for #1051). --- src/fx/fx.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/fx/fx.js b/src/fx/fx.js index 933c9bf4..59cdceac 100644 --- a/src/fx/fx.js +++ b/src/fx/fx.js @@ -143,7 +143,7 @@ jQuery.fn.extend({ * @see slideToggle(String|Number,Function) */ slideDown: function(speed,callback){ - return this.animate({height: "show"}, speed, callback); + return this.filter(":hidden").animate({height: "show"}, speed, callback).end(); }, /** @@ -168,7 +168,7 @@ jQuery.fn.extend({ * @see slideToggle(String|Number,Function) */ slideUp: function(speed,callback){ - return this.animate({height: "hide"}, speed, callback); + return this.filter(":visible").animate({height: "hide"}, speed, callback).end(); }, /** @@ -222,7 +222,7 @@ jQuery.fn.extend({ * @see fadeTo(String|Number,Number,Function) */ fadeIn: function(speed, callback){ - return this.animate({opacity: "show"}, speed, callback); + return this.filter(":hidden").animate({opacity: "show"}, speed, callback).end(); }, /** @@ -248,7 +248,7 @@ jQuery.fn.extend({ * @see fadeTo(String|Number,Number,Function) */ fadeOut: function(speed, callback){ - return this.animate({opacity: "hide"}, speed, callback); + return this.filter(":visible").animate({opacity: "hide"}, speed, callback).end(); }, /**