only calcs width when necessary

dont recalc width for every "priority" unless num columns hidden is different from last attempt. Saves computation especially because default priority =99!
mcuznz · Jul 6, 2014 · 619458f · 619458f
1 parent e2e5207
commit 619458f
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/jquery.magiccolumns.js b/jquery.magiccolumns.js
@@ -88,11 +88,13 @@
 
 		if (!lastpriority)
 			return;
-
-		while (lastpriority > 0 && ($(table).outerWidth() > $(table).parent().width())) {
 
-
-			var numHidden = 0;
+		var numHidden = 0;
+		var lastNumHidden=-1;
+		while (lastpriority > 0 && (numHidden == lastNumHidden  || ($(table).outerWidth() > $(table).parent().width()))) {
+
+			lastNumHidden = numHidden;
+
 			$(table).find('th').each(function() {
 				if ($(this).data('priority') == lastpriority) {
 					$(this).css({display: 'none'});
@@ -106,7 +108,7 @@
 					numHidden++;
 				}
 			});
-
+			
 			// walk down priority in case we're still too large
 			lastpriority -= 1;