From e1f16799bac16c568bc852ac9a39538c7eb91c24 Mon Sep 17 00:00:00 2001
From: Denozordec <shatskiy.blog@gmail.com>
Date: Tue, 31 Mar 2026 22:02:12 +0700
Subject: [PATCH] Refactor select component to enhance value binding and event
 handling

- Updated the alias filter select component to bind the selected value directly, simplifying the value change logic.
- Enhanced the select component to support an optional `onValueChange` callback, improving flexibility in handling value changes.
---
 web/src/lib/components/alias-filter-select.svelte |  8 +-------
 web/src/lib/components/ui/select/select.svelte    | 15 +++++++++++++--
 2 files changed, 14 insertions(+), 9 deletions(-)

diff --git a/web/src/lib/components/alias-filter-select.svelte b/web/src/lib/components/alias-filter-select.svelte
index 24057e9..9c9b43e 100644
--- a/web/src/lib/components/alias-filter-select.svelte
+++ b/web/src/lib/components/alias-filter-select.svelte
@@ -25,13 +25,7 @@
 
 <label class="text-xs text-muted-foreground">
 	{label}
-	<Select.Root
-		type="single"
-		value={value}
-		onValueChange={(next) => {
-			value = (next as string) || "all";
-		}}
-	>
+	<Select.Root type="single" bind:value>
 		<Select.Trigger class="mt-1 w-[220px]">
 			{selectedLabel}
 		</Select.Trigger>
diff --git a/web/src/lib/components/ui/select/select.svelte b/web/src/lib/components/ui/select/select.svelte
index 05eb663..5eba66b 100644
--- a/web/src/lib/components/ui/select/select.svelte
+++ b/web/src/lib/components/ui/select/select.svelte
@@ -4,8 +4,19 @@
 	let {
 		open = $bindable(false),
 		value = $bindable(),
+		onValueChange,
 		...restProps
-	}: SelectPrimitive.RootProps = $props();
+	}: SelectPrimitive.RootProps & {
+		onValueChange?: (next: string | string[]) => void;
+	} = $props();
 </script>
 
-<SelectPrimitive.Root bind:open bind:value={value as never} {...restProps} />
+<SelectPrimitive.Root
+	bind:open
+	bind:value={value as never}
+	onValueChange={(next: unknown) => {
+		value = next as typeof value;
+		onValueChange?.(next as string | string[]);
+	}}
+	{...restProps}
+/>