fix: dont use onMount inside CommandInput action for autofocus (#25)

Co-authored-by: Hunter Johnston <[email protected]>
huntabyte · Nov 18, 2023 · 30fac0b · 30fac0b
1 parent 4f271e4
commit 30fac0b
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 6 deletions.
diff --git a/.changeset/olive-lobsters-whisper.md b/.changeset/olive-lobsters-whisper.md
@@ -0,0 +1,5 @@
+---
+"cmdk-sv": patch
+---
+
+fix: `autofocus` input behavior 
diff --git a/src/lib/cmdk/components/CommandInput.svelte b/src/lib/cmdk/components/CommandInput.svelte
@@ -3,7 +3,6 @@
 	import { ITEM_SELECTOR, VALUE_ATTR, getCtx, getState } from '../command.js';
 	import { addEventListener, isBrowser, isHTMLInputElement } from '$lib/internal/index.js';
 	import type { InputEvents, InputProps } from '../types.js';
-	import { onMount } from 'svelte';
 	import { sleep } from '$lib/internal/helpers/sleep.js';
 
 	type $$Props = InputProps;
@@ -31,11 +30,10 @@
 	}
 
 	function action(node: HTMLInputElement) {
-		onMount(() => {
-			if (autofocus) {
-				sleep(10).then(() => node.focus());
-			}
-		});
+		if (autofocus) {
+			sleep(10).then(() => node.focus());
+		}
+
 		const unsubEvents = addEventListener(node, 'change', (e) => {
 			if (!isHTMLInputElement(e.target)) return;
 			state.updateState('search', e.target.value);