Merge branch 'develop' into issue/10525/user-avatat-delete-bug

rajku-dev · Feb 19, 2025 · 0dce732 · 0dce732
2 parents f7dcfc7 + fa57650
commit 0dce732
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 6 deletions.
diff --git a/src/components/Common/SearchByMultipleFields.tsx b/src/components/Common/SearchByMultipleFields.tsx
@@ -47,6 +47,7 @@ interface SearchByMultipleFieldsProps {
   clearSearch?: { value: boolean; params?: string[] };
   enableOptionButtons?: boolean;
   onFieldChange?: (options: SearchOption) => void;
+  autoFocus?: boolean;
 }
 
 const KeyboardShortcutHint = ({ open }: { open: boolean }) => {
@@ -90,6 +91,7 @@ const SearchByMultipleFields: React.FC<SearchByMultipleFieldsProps> = ({
   clearSearch,
   onFieldChange,
   enableOptionButtons = true,
+  autoFocus = false,
 }) => {
   const { t } = useTranslation();
   const [selectedOptionIndex, setSelectedOptionIndex] =
@@ -175,12 +177,6 @@ const SearchByMultipleFields: React.FC<SearchByMultipleFieldsProps> = ({
     return () => document.removeEventListener("keydown", handleKeyDown);
   }, [focusedIndex, open, handleOptionChange, options]);
 
-  useEffect(() => {
-    if (inputRef.current) {
-      inputRef.current.focus();
-    }
-  }, [selectedOptionIndex]);
-
   useEffect(() => {
     if (selectedOption.value !== searchValue) {
       onSearch(selectedOption.key, searchValue);
@@ -224,6 +220,12 @@ const SearchByMultipleFields: React.FC<SearchByMultipleFieldsProps> = ({
     }
   }, [selectedOption, searchValue, t, inputClassName, open]);
 
+  useEffect(() => {
+    if (autoFocus) {
+      inputRef.current?.focus();
+    }
+  }, [autoFocus, open, selectedOptionIndex]);
+
   return (
     <div
       className={cn(

diff --git a/src/components/Resource/ResourceList.tsx b/src/components/Resource/ResourceList.tsx
@@ -154,6 +154,7 @@ export default function ResourceList({ facilityId }: { facilityId: string }) {
                           })
                         }
                         className="w-full border-none shadow-none"
+                        autoFocus
                       />
                     </div>
                   </PopoverContent>

diff --git a/src/pages/Encounters/EncounterList.tsx b/src/pages/Encounters/EncounterList.tsx
@@ -261,6 +261,7 @@ export function EncounterList({
                         onFieldChange={handleFieldChange}
                         onSearch={handleSearch}
                         className="w-full border-none shadow-none"
+                        autoFocus
                       />
                     </div>
                   </PopoverContent>
-Original file line number
+Diff line change
@@ Expand Up @@
                               })
                             }
                             className="w-full border-none shadow-none"
+                            autoFocus
                           />
                         </div>
                       </PopoverContent>
@@ Expand Down @@