only add 'split' to the node in the next column if part of it ended up in the current column

adamwulf · adamwulf · commit 7fbc54b468a9 · 2013-11-22T13:02:09.000-06:00
diff --git a/src/jquery.columnizer.js b/src/jquery.columnizer.js
@@ -289,7 +289,7 @@
 						// it in half, leaving some of it in pullOutHere
 						$clone.empty();
 						if(!columnize($clone, $cloneMe, $parentColumn, targetHeight)){
-							// this node still has non-text nodes to split
+							// this node may still have non-text nodes to split
 							// add the split class and then recur
 							$cloneMe.addClass(prefixTheClassName("split"));
 							
@@ -310,6 +310,7 @@
 						if($clone.get(0).childNodes.length === 0){
 							// it was split, but nothing is in it :(
 							$clone.remove();
+							$cloneMe.removeClass(prefixTheClassName("split"));
 						}
 					}
 				}

Original file line number	Diff line number	Diff line change
`@@ -289,7 +289,7 @@`
`289`	`289`	`// it in half, leaving some of it in pullOutHere`
`290`	`290`	`$clone.empty();`
`291`	`291`	`if(!columnize($clone, $cloneMe, $parentColumn, targetHeight)){`
`292`		`- // this node still has non-text nodes to split`
	`292`	`+ // this node may still have non-text nodes to split`
`293`	`293`	`// add the split class and then recur`
`294`	`294`	`$cloneMe.addClass(prefixTheClassName("split"));`
`295`	`295`
`@@ -310,6 +310,7 @@`
`310`	`310`	`if($clone.get(0).childNodes.length === 0){`
`311`	`311`	`// it was split, but nothing is in it :(`
`312`	`312`	`$clone.remove();`
	`313`	`+ $cloneMe.removeClass(prefixTheClassName("split"));`
`313`	`314`	`}`
`314`	`315`	`}`
`315`	`316`	`}`