🌸 Add warn and new assertion when rows=0 and collision !== 'none (#86)

Author: cuire

src/lib/Grid.svelte

@@ -87,7 +87,7 @@
 		xs: 320
 	};
 
-	$: assertGridOptions({ cols, rows, itemSize });
+	$: assertGridOptions({ cols, rows, itemSize, collision });
 
 	/**
 	 * Bound the grid items to the grid container.
@@ -175,7 +175,7 @@
 	}
 
 	$: if (collision !== 'none') {
-		rows = 0;
+		_rows = 0;
 	}
 
 	/**

src/lib/utils/assert.ts

@@ -1,13 +1,25 @@
-import type { GridSize, ItemSize } from '$lib/types';
+import type { Collision, GridSize, ItemSize } from '$lib/types';
 
 export type GridOptions = {
 	cols: GridSize;
 	rows: GridSize;
 	itemSize: Partial<ItemSize>;
+	collision?: Collision;
 };
 
 export function assertGridOptions(options: GridOptions) {
-	const { cols, rows, itemSize } = options;
+	const { cols, rows, itemSize, collision } = options;
+
+	if (rows !== 0 && collision && collision !== 'none') {
+		console.warn('`rows` is ignored and setted to 0 when `collision` is not `none`');
+	}
+
+	if (collision && collision !== 'none' && itemSize?.height === undefined) {
+		throw new Error(
+			'If `collision` is not `none`, the `itemSize.height` parameter must be specified'
+		);
+	}
+
 	if (
 		(cols === 0 && itemSize?.width === undefined) ||
 		(typeof cols === 'object' && Object.values(cols).includes(0) && itemSize?.width === undefined)