fix(Listbox): re-assign modelValue to trigger the update event (#1864)

kricsleo · web-flow · commit c12ea53a5651 · 2025-05-05T23:39:14.000+08:00
diff --git a/packages/core/src/shared/useSelectionBehavior.ts b/packages/core/src/shared/useSelectionBehavior.ts
@@ -18,9 +18,9 @@ export function useSelectionBehavior<T>(
       else {
         const index = modelValue.value.findIndex(v => condition(v))
         if (index !== -1)
-          modelValue.value.splice(index, 1)
+          modelValue.value = modelValue.value.filter((_, i) => i !== index)
         else
-          modelValue.value.push(val)
+          modelValue.value = [...modelValue.value, val]
       }
     }
     // single select

Original file line number	Diff line number	Diff line change
`@@ -18,9 +18,9 @@ export function useSelectionBehavior<T>(`
`18`	`18`	`else {`
`19`	`19`	`const index = modelValue.value.findIndex(v => condition(v))`
`20`	`20`	`if (index !== -1)`
`21`		`- modelValue.value.splice(index, 1)`
	`21`	`+ modelValue.value = modelValue.value.filter((_, i) => i !== index)`
`22`	`22`	`else`
`23`		`- modelValue.value.push(val)`
	`23`	`+ modelValue.value = [...modelValue.value, val]`
`24`	`24`	`}`
`25`	`25`	`}`
`26`	`26`	`// single select`