Merge pull request #54962 from daledah/fix/54931

[CP Staging] fix: unable to change currency with arrow key
Expensify · Jan 8, 2025 · 9589f64 · 9589f64
2 parents cc8fcc3 + 8924bc7
commit 9589f64
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/components/SelectionList/BaseSelectionList.tsx b/src/components/SelectionList/BaseSelectionList.tsx
@@ -333,14 +333,15 @@ function BaseSelectionList<TItem extends ListItem>(
         isFocused,
     });
 
+    const selectedItemIndex = useMemo(() => flattenedSections.allOptions.findIndex((option) => option.isSelected), [flattenedSections.allOptions]);
+
     useEffect(() => {
-        const selectedItemIndex = flattenedSections.allOptions.findIndex((option) => option.isSelected);
         if (selectedItemIndex === -1 || selectedItemIndex === focusedIndex) {
             return;
         }
         setFocusedIndex(selectedItemIndex);
         // eslint-disable-next-line react-compiler/react-compiler, react-hooks/exhaustive-deps
-    }, [flattenedSections]);
+    }, [selectedItemIndex]);
 
     const clearInputAfterSelect = useCallback(() => {
         onChangeText?.('');