Merge pull request #145 from Neovici/feat/query-case

feat(autocomplete): maintain query case
Neovici · Jun 26, 2023 · f65df29 · f65df29
2 parents c386b18 + c69ea4b
commit f65df29
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/src/autocomplete/use-autocomplete.ts b/src/autocomplete/use-autocomplete.ts
@@ -65,7 +65,7 @@ export const useAutocomplete = <I>({
 		),
 		{ active, focused, onFocus, setClosed } = useFocus(thru),
 		empty = !text,
-		query = useMemo(() => text?.trim().toLowerCase(), [text]),
+		query = useMemo(() => text?.trim(), [text]),
 		host = useHost(),
 		onText = useNotify(host, _onText, 'text'),
 		onChange = useCallback(

diff --git a/src/autocomplete/util.ts b/src/autocomplete/util.ts
@@ -5,9 +5,10 @@ export const search = <I>(
 	query: string,
 	textual: (i: I) => string
 ) => {
+	const qry = query.toLowerCase();
 	const matches = [];
 	for (const item of source) {
-		const index = textual(item).toLowerCase().indexOf(query);
+		const index = textual(item).toLowerCase().indexOf(qry);
 		if (index < 0) {
 			continue;
 		}

diff --git a/test/use-autocomplete.test.js b/test/use-autocomplete.test.js
@@ -21,7 +21,7 @@ describe('use-autocomplete', () => {
 					.textProperty=${'text'}
 				/>
 			`);
-		expect(result.current.query).to.equal('it');
+		expect(result.current.query).to.equal('It');
 		expect(await result.current.items$).to.be.empty;
 	});
 
@@ -123,7 +123,7 @@ describe('use-autocomplete', () => {
 					.external=${true}
 				/>
 			`);
-		expect(result.current.query).to.equal('la');
+		expect(result.current.query).to.equal('La');
 		result.current.onFocus({ currentTarget: { matches: () => true } });
 		await nextFrame();