Finally fixed some child depth bugs relating to a poor understanding of the selector API

lucaswoj · lucaswoj · commit e75be5dc5dcd · 2010-12-25T17:42:05.000-05:00
diff --git a/jquery.nestedsortable.js b/jquery.nestedsortable.js
@@ -66,7 +66,6 @@ $.fn.nestedSortable = function(settings) {
 	                            }
 	                        });
 	                        $this.data("maxChildDepth", maxChildDepth);
-							console.log("Max Child Depth: " + maxChildDepth);
 						}
 						
 						$this.removeClass("longnonsensicalclassnamethatnobodywouldeveruse");
@@ -163,7 +162,8 @@ $.fn.nestedSortable = function(settings) {
                     stop: function(event, ui) {
 	
                         // Replace the placeholder with the original
-                        $placeholder.after($this.show()).remove();
+                        $placeholder.after($this.show());
+						$placeholder.remove();
 
 						// Re-enable text selection
 						if (settings.disableSelect) {

Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,6 @@ $.fn.nestedSortable = function(settings) {`
`66`	`66`	`}`
`67`	`67`	`});`
`68`	`68`	`$this.data("maxChildDepth", maxChildDepth);`
`69`		`- console.log("Max Child Depth: " + maxChildDepth);`
`70`	`69`	`}`
`71`	`70`
`72`	`71`	`$this.removeClass("longnonsensicalclassnamethatnobodywouldeveruse");`
`@@ -163,7 +162,8 @@ $.fn.nestedSortable = function(settings) {`
`163`	`162`	`stop: function(event, ui) {`
`164`	`163`
`165`	`164`	`// Replace the placeholder with the original`
`166`		`- $placeholder.after($this.show()).remove();`
	`165`	`+ $placeholder.after($this.show());`
	`166`	`+ $placeholder.remove();`
`167`	`167`
`168`	`168`	`// Re-enable text selection`
`169`	`169`	`if (settings.disableSelect) {`