Effects: rename effect.removePlaceholder to effect.cleanUpPlaceholder

Author: mikesherov

ui/effect-blind.js

@@ -55,7 +55,7 @@ $.effects.define( "blind", "hide", function( o, done ) {
 		easing: o.easing,
 		complete: function() {
 
-			$.effects.removePlaceholder( placeholder, el );
+			$.effects.cleanUpPlaceholder( placeholder, el );
 
 			if ( !show ) {
 				el.hide();

ui/effect-bounce.js

@@ -95,7 +95,7 @@ $.effects.define( "bounce", function( o, done ) {
 	}
 
 	el.queue(function() {
-		$.effects.removePlaceholder( placeholder, el );
+		$.effects.cleanUpPlaceholder( placeholder, el );
 
 		if ( hide ) {
 			el.hide();

ui/effect-clip.js

@@ -54,7 +54,7 @@ $.effects.define( "clip", "hide", function( o, done ) {
 		easing: o.easing,
 		complete: function() {
 
-			$.effects.removePlaceholder( placeholder, el );
+			$.effects.cleanUpPlaceholder( placeholder, el );
 
 			if ( !show ) {
 				el.hide();

ui/effect-drop.js

@@ -55,7 +55,7 @@ $.effects.define( "drop", "hide", function( o, done ) {
 		duration: o.duration,
 		easing: o.easing,
 		complete: function() {
-			$.effects.removePlaceholder( placeholder, el );
+			$.effects.cleanUpPlaceholder( placeholder, el );
 
 			if ( mode === "hide" ) {
 				el.hide();

ui/effect-fold.js

@@ -70,7 +70,7 @@ $.effects.define( "fold", "hide", function( o, done ) {
 		.animate( animation1, duration, o.easing )
 		.animate( animation2, duration, o.easing )
 		.queue(function() {
-			$.effects.removePlaceholder( placeholder, el );
+			$.effects.cleanUpPlaceholder( placeholder, el );
 
 			if ( hide ) {
 				el.hide();

ui/effect-shake.js

@@ -54,12 +54,13 @@ $.effects.define( "shake", function( o, done ) {
 	for ( ; i < times; i++ ) {
 		el.animate( animation1, speed, o.easing ).animate( animation2, speed, o.easing );
 	}
+
 	el
 		.animate( animation1, speed, o.easing )
 		.animate( animation, speed / 2, o.easing )
 		.queue(function() {
 
-			$.effects.removePlaceholder( placeholder, el );
+			$.effects.cleanUpPlaceholder( placeholder, el );
 
 			if ( o.mode === "hide" ) {
 				el.hide();

ui/effect-size.js

@@ -180,7 +180,7 @@ $.effects.define( "size", function( o, done ) {
 			var offset = el.offset();
 
 			if ( restore ) {
-				$.effects.removePlaceholder( placeholder, el );
+				$.effects.cleanUpPlaceholder( placeholder, el );
 			}
 
 			if ( to.opacity === 0 ) {
@@ -191,10 +191,7 @@ $.effects.define( "size", function( o, done ) {
 			}
 
 			if ( !restore ) {
-
-				if ( placeholder ) {
-					placeholder.remove();
-				}
+				$.effects.removePlaceholder( placeholder );
 
 				el.css("position", position === "static" ? "relative" : position );
 

ui/effect-slide.js

@@ -60,7 +60,7 @@ $.effects.define( "slide", "show", function( o, done ) {
 		duration: o.duration,
 		easing: o.easing,
 		complete: function() {
-			$.effects.removePlaceholder( placeholder, el );
+			$.effects.cleanUpPlaceholder( placeholder, el );
 
 			if ( mode === "hide" ) {
 				el.hide();

ui/effect.js

@@ -1151,16 +1151,19 @@ $.extend( $.effects, {
 		return placeholder;
 	},
 
-	// removes a placeholder if it exists and restores
-	// properties that were modified during placeholder creation
-	removePlaceholder: function( placeholder, el ) {
-		$.effects.restoreStyle( el );
-
+	removePlaceholder: function( placeholder ) {
 		if ( placeholder ) {
 			placeholder.remove();
 		}
 	},
 
+	// removes a placeholder if it exists and restores
+	// properties that were modified during placeholder creation
+	cleanUpPlaceholder: function( placeholder, el ) {
+		$.effects.restoreStyle( el );
+		$.effects.removePlaceholder( placeholder );
+	},
+
 	setTransition: function( element, list, factor, value ) {
 		value = value || {};
 		$.each( list, function( i, x ) {