fix perf-test

Expensify · Oct 10, 2024 · 4e20be9 · 4e20be9
1 parent c61aa1a
commit 4e20be9
Showing 1 changed file with 9 additions and 9 deletions.
diff --git a/src/pages/home/report/ReportActionCompose/AttachmentPickerWithMenuItems.tsx b/src/pages/home/report/ReportActionCompose/AttachmentPickerWithMenuItems.tsx
@@ -23,6 +23,7 @@ import Navigation from '@libs/Navigation/Navigation';
 import * as ReportUtils from '@libs/ReportUtils';
 import * as SubscriptionUtils from '@libs/SubscriptionUtils';
 import * as IOU from '@userActions/IOU';
+import * as Modal from '@userActions/Modal';
 import * as Report from '@userActions/Report';
 import * as Task from '@userActions/Task';
 import DelegateNoAccessModal from '@src/components/DelegateNoAccessModal';
@@ -229,13 +230,6 @@ function AttachmentPickerWithMenuItems({
                     {
                         icon: Expensicons.Paperclip,
                         text: translate('reportActionCompose.addAttachment'),
-                        onSelected: () => {
-                            if (Browser.isSafari()) {
-                                return;
-                            }
-                            triggerAttachmentPicker();
-                        },
-                        shouldCallAfterModalHide: true,
                     },
                 ];
                 return (
@@ -322,8 +316,14 @@ function AttachmentPickerWithMenuItems({
                                 // In order for the file picker to open dynamically, the click
                                 // function must be called from within a event handler that was initiated
                                 // by the user on Safari.
-                                if (index === menuItems.length - 1 && Browser.isSafari()) {
-                                    triggerAttachmentPicker();
+                                if (index === menuItems.length - 1) {
+                                    if (Browser.isSafari()) {
+                                        triggerAttachmentPicker();
+                                        return;
+                                    }
+                                    Modal.close(() => {
+                                        triggerAttachmentPicker();
+                                    });
                                 }
                             }}
                             anchorPosition={styles.createMenuPositionReportActionCompose(shouldUseNarrowLayout, windowHeight, windowWidth)}