_grid-framework – replace #for- mixin loops with each(range())

Author: calvinjuarez

less/mixins/_grid-framework.less

@@ -81,13 +81,11 @@
 	each(keys(@breakpoints), #(@breakpoint) {
 		@infix: breakpoint-infix(@breakpoint, @breakpoints);
 
-		#for-1-to-columns(@i: 1) when (@i <= @columns) {
+		each(range(@columns), #(@i) {
 			.col@{infix}-@{i} {
 				&:extend(.grid-column);
 			}
-
-			#for-1-to-columns((@i + 1));
-		} #for-1-to-columns();
+		});
 
 		.col@{infix},
 		.col@{infix}-auto {
@@ -107,35 +105,29 @@
 				max-width: none; // Reset earlier grid tiers
 			}
 
-			#for-1-to-columns-col(@ii: 1) when (@ii <= @columns) {
-				.col@{infix}-@{ii} {
-					#make-col(@ii, @columns);
+			each(range(@columns), #(@i) {
+				.col@{infix}-@{i} {
+					#make-col(@i, @columns);
 				}
-
-				#for-1-to-columns-col((@ii + 1));
-			} #for-1-to-columns-col();
+			});
 
 			.order@{infix}-first { order: -1; }
 
 			.order@{infix}-last { order: (@columns + 1); }
 
-			#for-1-to-columns-order(@ii: 0) when (@ii <= @columns) {
-				.order@{infix}-@{ii} { order: @ii; }
+			each(range(0, @columns), #(@i) {
+				.order@{infix}-@{i} { order: @i; }
+			});
 
-				#for-1-to-columns-order((@ii + 1));
-			} #for-1-to-columns-order();
-
-			// `@ii < @columns` because offsetting by the width of an entire row isn't possible
-			#for-1-to-columns-offset(@ii: 0) when (@ii < @columns) {
-				& when not (@ii = 0),
-				(@ii = 0) and not (@infix = ~"") {
-					.offset@{infix}-@{ii} {
-						#make-col-offset(@ii, @columns);
+			// `@columns - 1` because offsetting by the width of an entire row isn't possible
+			each(range(0, (@columns - 1)), #(@i) {
+				& when not (@i = 0),
+				(@i = 0) and not (@infix = ~"") {
+					.offset@{infix}-@{i} {
+						#make-col-offset(@i, @columns);
 					}
 				}
-
-				#for-1-to-columns-offset((@ii + 1));
-			} #for-1-to-columns-offset();
+			});
 		});
 	});
 }