From 6f41577ec91281ab0475c04d661735a605806ec8 Mon Sep 17 00:00:00 2001 From: Aman Harwara Date: Mon, 14 Mar 2022 21:47:42 +0530 Subject: [PATCH] fix: make search case-insensitive (#925) --- .../AttachedFilesPopover/AttachedFilesPopover.tsx | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/app/assets/javascripts/components/AttachedFilesPopover/AttachedFilesPopover.tsx b/app/assets/javascripts/components/AttachedFilesPopover/AttachedFilesPopover.tsx index 48b3f034b..b5e2dfd7a 100644 --- a/app/assets/javascripts/components/AttachedFilesPopover/AttachedFilesPopover.tsx +++ b/app/assets/javascripts/components/AttachedFilesPopover/AttachedFilesPopover.tsx @@ -5,7 +5,13 @@ import { ContentType, SNFile, SNNote } from '@standardnotes/snjs'; import { FilesIllustration } from '@standardnotes/stylekit'; import { observer } from 'mobx-react-lite'; import { FunctionComponent } from 'preact'; -import { StateUpdater, useCallback, useEffect, useState } from 'preact/hooks'; +import { + StateUpdater, + useCallback, + useEffect, + useRef, + useState, +} from 'preact/hooks'; import { Button } from '../Button'; import { Icon } from '../Icon'; import { PopoverFileItem } from './PopoverFileItem'; @@ -44,6 +50,7 @@ export const AttachedFilesPopover: FunctionComponent = observer( const [attachedFiles, setAttachedFiles] = useState([]); const [allFiles, setAllFiles] = useState([]); const [searchQuery, setSearchQuery] = useState(''); + const searchInputRef = useRef(null); const filesList = currentTab === PopoverTabs.AttachedFiles ? attachedFiles : allFiles; @@ -51,7 +58,8 @@ export const AttachedFilesPopover: FunctionComponent = observer( const filteredList = searchQuery.length > 0 ? filesList.filter( - (file) => file.name.toLowerCase().indexOf(searchQuery) !== -1 + (file) => + file.name.toLowerCase().indexOf(searchQuery.toLowerCase()) !== -1 ) : filesList; @@ -151,12 +159,14 @@ export const AttachedFilesPopover: FunctionComponent = observer( setSearchQuery((e.target as HTMLInputElement).value); }} onBlur={closeOnBlur} + ref={searchInputRef} /> {searchQuery.length > 0 && (