From f274c9900f7b1feb80ee8cfd3cd3c5784daf9016 Mon Sep 17 00:00:00 2001
From: Sam Chau <chaucted@gmail.com>
Date: Wed, 5 Nov 2025 07:35:48 +1030
Subject: [PATCH] refactor(table): move Column interface to separate types file
 for better organization

---
 src/lib/client/ui/table/Table.svelte          | 22 +------------------
 src/lib/client/ui/table/types.ts              | 21 ++++++++++++++++++
 src/routes/databases/+page.svelte             |  2 +-
 .../[databaseId]/views/TableView.svelte       |  3 ++-
 4 files changed, 25 insertions(+), 23 deletions(-)
 create mode 100644 src/lib/client/ui/table/types.ts

diff --git a/src/lib/client/ui/table/Table.svelte b/src/lib/client/ui/table/Table.svelte
index 4c59f64..8c5d563 100644
--- a/src/lib/client/ui/table/Table.svelte
+++ b/src/lib/client/ui/table/Table.svelte
@@ -1,25 +1,5 @@
 <script lang="ts" generics="T extends Record<string, any>">
-	import type { ComponentType } from 'svelte';
-
-	/**
-	 * Column definition for table
-	 */
-	export interface Column<T> {
-		/** Unique key for the column */
-		key: string;
-		/** Header text to display */
-		header: string;
-		/** Optional icon component to display before header text */
-		headerIcon?: ComponentType;
-		/** Optional width class (e.g., 'w-32', 'w-1/4') */
-		width?: string;
-		/** Text alignment */
-		align?: 'left' | 'center' | 'right';
-		/** Whether column is sortable */
-		sortable?: boolean;
-		/** Custom cell renderer - receives the full row object */
-		cell?: (row: T) => string | ComponentType | { html: string };
-	}
+	import type { Column } from './types';
 
 	/**
 	 * Props
diff --git a/src/lib/client/ui/table/types.ts b/src/lib/client/ui/table/types.ts
new file mode 100644
index 0000000..8758873
--- /dev/null
+++ b/src/lib/client/ui/table/types.ts
@@ -0,0 +1,21 @@
+import type { ComponentType } from 'svelte';
+
+/**
+ * Column definition for table
+ */
+export interface Column<T> {
+	/** Unique key for the column */
+	key: string;
+	/** Header text to display */
+	header: string;
+	/** Optional icon component to display before header text */
+	headerIcon?: ComponentType;
+	/** Optional width class (e.g., 'w-32', 'w-1/4') */
+	width?: string;
+	/** Text alignment */
+	align?: 'left' | 'center' | 'right';
+	/** Whether column is sortable */
+	sortable?: boolean;
+	/** Custom cell renderer - receives the full row object */
+	cell?: (row: T) => string | ComponentType | { html: string };
+}
diff --git a/src/routes/databases/+page.svelte b/src/routes/databases/+page.svelte
index 6b69664..a02d94a 100644
--- a/src/routes/databases/+page.svelte
+++ b/src/routes/databases/+page.svelte
@@ -7,7 +7,7 @@
 	import Modal from '$ui/modal/Modal.svelte';
 	import { alertStore } from '$alerts/store';
 	import type { PageData } from './$types';
-	import type { Column } from '$ui/table/Table.svelte';
+	import type { Column } from '$ui/table/types';
 	import type { DatabaseInstance } from '$db/queries/databaseInstances.ts';
 
 	export let data: PageData;
diff --git a/src/routes/quality-profiles/[databaseId]/views/TableView.svelte b/src/routes/quality-profiles/[databaseId]/views/TableView.svelte
index 93fa043..56b0065 100644
--- a/src/routes/quality-profiles/[databaseId]/views/TableView.svelte
+++ b/src/routes/quality-profiles/[databaseId]/views/TableView.svelte
@@ -1,5 +1,6 @@
 <script lang="ts">
-	import Table, { type Column } from '$ui/table/Table.svelte';
+	import Table from '$ui/table/Table.svelte';
+	import type { Column } from '$ui/table/types';
 	import type { QualityProfileTableRow } from '$lib/server/pcd/queries/qualityProfiles';
 	import { Tag, FileText, Layers, BookOpenText, Gauge, Earth } from 'lucide-svelte';
 	import { goto } from '$app/navigation';