X-Git-Url: http://git.asbjorn.biz/?a=blobdiff_plain;f=src%2Ffx%2Ffx.js;h=d83bf0dd431075fcdbe5428ca76ec3ba4e756819;hb=bfa79591b3c849d8df6cd8c7eac757732a855519;hp=6dd3279d89ab645be4115f55c596c2b8bd16557d;hpb=0798c6e64e4e0c984d5ece50ade61827c2105c39;p=jquery.git

diff --git a/src/fx/fx.js b/src/fx/fx.js
index 6dd3279..d83bf0d 100644
--- a/src/fx/fx.js
+++ b/src/fx/fx.js
@@ -33,16 +33,18 @@ jQuery.fn.extend({
 	 * @see hide(String|Number,Function)
 	 */
 	show: function(speed,callback){
-		return speed ?
-			this.animate({
+		var hidden = this.filter(":hidden");
+		speed ?
+			hidden.animate({
 				height: "show", width: "show", opacity: "show"
 			}, speed, callback) :
 			
-			this.each(function(){
+			hidden.each(function(){
 				this.style.display = this.oldblock ? this.oldblock : "";
 				if ( jQuery.css(this,"display") == "none" )
 					this.style.display = "block";
 			});
+		return this;
 	},
 	
 	/**
@@ -78,17 +80,19 @@ jQuery.fn.extend({
 	 * @see show(String|Number,Function)
 	 */
 	hide: function(speed,callback){
-		return speed ?
-			this.animate({
+		var visible = this.filter(":visible");
+		speed ?
+			visible.animate({
 				height: "hide", width: "hide", opacity: "hide"
 			}, speed, callback) :
 			
-			this.each(function(){
+			visible.each(function(){
 				this.oldblock = this.oldblock || jQuery.css(this,"display");
 				if ( this.oldblock == "none" )
 					this.oldblock = "block";
 				this.style.display = "none";
 			});
+		return this;
 	},
 
 	// Save the old toggle function
@@ -352,14 +356,14 @@ jQuery.fn.extend({
 jQuery.extend({
 	
 	speed: function(speed, easing, fn) {
-		var opt = speed.constructor == Object ? speed : {
+		var opt = speed && speed.constructor == Object ? speed : {
 			complete: fn || !fn && easing || 
-				speed.constructor == Function && speed,
+				speed && speed.constructor == Function && speed,
 			duration: speed,
 			easing: fn && easing || easing && easing.constructor != Function && easing
 		};
 
-		opt.duration = (opt.duration.constructor == Number ? 
+		opt.duration = (opt.duration && opt.duration.constructor == Number ? 
 			opt.duration : 
 			{ slow: 600, fast: 200 }[opt.duration]) || 400;