Effects.*: Updating Effect Method API to avoid duplicating the queue call - Fixes #7318 - Add the queue functions to $.fn.effect()

1 files changed, 30 insertions, 33 deletions

diff --git a/ui/jquery.effects.transfer.js b/ui/jquery.effects.transfer.js
index f0f9d9fd4..061882fec 100644
--- a/ui/jquery.effects.transfer.js
+++ b/ui/jquery.effects.transfer.js
@@ -12,39 +12,36 @@
  */
 (function( $, undefined ) {
 
-$.effects.effect.transfer = function( o ) {
-	
-	return this.queue( function() {
-		var elem = $( this ),
-			target = $( o.to ),
-			targetFixed = target.css( "position" ) === "fixed",
-			body = $("body"),
-			fixTop = targetFixed ? body.scrollTop() : 0,
-			fixLeft = targetFixed ? body.scrollLeft() : 0,
-			endPosition = target.offset(),
-			animation = {
-				top: endPosition.top - fixTop ,
-				left: endPosition.left - fixLeft ,
-				height: target.innerHeight(),
-				width: target.innerWidth()
-			},
-			startPosition = elem.offset(),
-			transfer = $( '<div class="ui-effects-transfer"></div>' )
-				.appendTo( document.body )
-				.addClass( o.className )
-				.css({
-					top: startPosition.top - fixTop ,
-					left: startPosition.left - fixLeft ,
-					height: elem.innerHeight(),
-					width: elem.innerWidth(),
-					position: targetFixed ? "fixed" : "absolute"
-				})
-				.animate( animation, o.duration, o.easing, function() {
-					transfer.remove();
-					$.isFunction( o.complete ) && o.complete.apply(elem[0], arguments);
-					elem.dequeue();
-				});
-	});
+$.effects.effect.transfer = function( o, next ) {
+	var elem = $( this ),
+		target = $( o.to ),
+		targetFixed = target.css( "position" ) === "fixed",
+		body = $("body"),
+		fixTop = targetFixed ? body.scrollTop() : 0,
+		fixLeft = targetFixed ? body.scrollLeft() : 0,
+		endPosition = target.offset(),
+		animation = {
+			top: endPosition.top - fixTop ,
+			left: endPosition.left - fixLeft ,
+			height: target.innerHeight(),
+			width: target.innerWidth()
+		},
+		startPosition = elem.offset(),
+		transfer = $( '<div class="ui-effects-transfer"></div>' )
+			.appendTo( document.body )
+			.addClass( o.className )
+			.css({
+				top: startPosition.top - fixTop ,
+				left: startPosition.left - fixLeft ,
+				height: elem.innerHeight(),
+				width: elem.innerWidth(),
+				position: targetFixed ? "fixed" : "absolute"
+			})
+			.animate( animation, o.duration, o.easing, function() {
+				transfer.remove();
+				$.isFunction( o.complete ) && o.complete.apply(elem[0], arguments);
+				next();
+			});
 };
 
 })(jQuery);