fix: remove useMemo

Expensify · Feb 14, 2024 · cfc6174 · cfc6174
1 parent d738e43
commit cfc6174
Showing 1 changed file with 14 additions and 18 deletions.
diff --git a/src/components/Attachments/AttachmentCarousel/Pager/index.tsx b/src/components/Attachments/AttachmentCarousel/Pager/index.tsx
@@ -137,24 +137,20 @@ function AttachmentCarouselPager({items, activeSource, initialPage, onPageSelect
         [],
     );
 
-    const carouselItems = useMemo(
-        () =>
-            items.map((item, index) => (
-                <View
-                    key={item.source}
-                    style={styles.flex1}
-                >
-                    <CarouselItem
-                        // @ts-expect-error TODO: Remove this once AttachmentView (https://github.com/Expensify/App/issues/25150) is migrated to TypeScript.
-                        item={item}
-                        isSingleItem={items.length === 1}
-                        index={index}
-                        isFocused={index === activePageIndex && activeSource === item.source}
-                    />
-                </View>
-            )),
-        [activePageIndex, activeSource, items, styles.flex1],
-    );
+    const carouselItems = items.map((item, index) => (
+        <View
+            key={item.source}
+            style={styles.flex1}
+        >
+            <CarouselItem
+                // @ts-expect-error TODO: Remove this once AttachmentView (https://github.com/Expensify/App/issues/25150) is migrated to TypeScript.
+                item={item}
+                isSingleItem={items.length === 1}
+                index={index}
+                isFocused={index === activePageIndex && activeSource === item.source}
+            />
+        </View>
+    ));
 
     return (
         <AttachmentCarouselPagerContext.Provider value={contextValue}>