fixed whitespace text node causing fake dontend when it was nested deeply in what would be the last node of a column

Author: adamwulf

samples/issue_36.html

@@ -12,6 +12,7 @@
 		body { font-size: 9pt; }
 		#wrapper { margin: 0px 100px; }
 		.wide, .thin { clear:both; }
+		#overflow { clear:both; }
 	</style>
 	<script type="text/javascript" src="http://ajax.googleapis.com/ajax/libs/jquery/1.6.2/jquery.min.js"></script>
 	<script src="../src/jquery.columnizer.js" type="text/javascript" charset="utf-8"></script>
@@ -74,7 +75,7 @@ <h1>Soleat civibus in pri</h1>
 		<p>Per ne solum vivendo, fabulas dolorem vivendo in pro. Nec duis ignota cotidieque no, an per possit nostrum. Pro detraxit definitionem eu. Vivendo officiis no nam, eu has reque maiestatis percipitur, dolore reprimique accommodare cum ad. No utinam voluptua oportere pri, augue sonet dicant ei sea.</p>
 		<p>Sit et, an ius nihil apeirian. Eu posse tempor iuvaret cum. No diam dolor sea, postea mnesarchum ne ius, vel no utinam ignota dolores. Malis suscipit accusamus his ne, utinam assentior prodesset ea eam, facer partem antiopam et cum.</p>
 	</div>
-	<div class="overflow">
+	<div id="overflow">
 	</div>
 
 </div>

samples/issue_44.html

@@ -9,7 +9,9 @@
 	<meta name="author" content="Matt Wilson">
 	<!-- Date: 2008-07-12 -->
 	<style>
-		.column *{ padding: 5px; }
+		.column div{ padding: 5px; }
+		.last.column div{ padding-right: 0px; }
+		.first.column div{ padding-left: 0px; }
 		.columnas { width: 100%; }
 		p { text-indent: 10pt; }
 		p.split { text-indent: 0pt; }

src/jquery.columnizer.js

@@ -296,7 +296,7 @@
 					split($col, $destroyable, $col, targetHeight);
 				}
 
-				while(checkDontEndColumn($col.contents(":last").length && $col.contents(":last").get(0))){
+				while($col.contents(":last").length && checkDontEndColumn($col.contents(":last").get(0))){
 					var $lastKid = $col.contents(":last");
 					$lastKid.remove();
 					$destroyable.prepend($lastKid);
@@ -306,9 +306,11 @@
 				var div = document.createElement('DIV');
 				while($destroyable[0].childNodes.length > 0){
 					var kid = $destroyable[0].childNodes[0];
-					for(var i=0;i<kid.attributes.length;i++){
-						if(kid.attributes[i].nodeName.indexOf("jQuery") == 0){
-							kid.removeAttribute(kid.attributes[i].nodeName);
+					if(kid.attributes){
+						for(var i=0;i<kid.attributes.length;i++){
+							if(kid.attributes[i].nodeName.indexOf("jQuery") == 0){
+								kid.removeAttribute(kid.attributes[i].nodeName);
+							}
 						}
 					}
 					div.innerHTML = "";
@@ -339,7 +341,12 @@
 				// text node. ensure that the text
 				// is not 100% whitespace
 				if(/^\s+$/.test(dom.nodeValue)){
-					return true;
+						//
+				        // ok, it's 100% whitespace,
+				        // so we should return checkDontEndColumn
+				        // of the inputs previousSibling
+				        if(!dom.previousSibling) return false;
+					return checkDontEndColumn(dom.previousSibling);
 				}
 				return false;
 			}
@@ -434,7 +441,7 @@
 //						alert("not splitting a dontend");
 					}
 
-					while(checkDontEndColumn($col.contents(":last").length && $col.contents(":last").get(0))){
+					while($col.contents(":last").length && checkDontEndColumn($col.contents(":last").get(0))){
 						var $lastKid = $col.contents(":last");
 						$lastKid.remove();
 						$destroyable.prepend($lastKid);