Merge pull request #27649 from ishpaul777/fix/removeDragoverListner-i…

…n-FAB-menu Fix remove dragover listner when popover hides
Expensify · Sep 21, 2023 · cfc4239 · cfc4239
2 parents 0c80655 + e837797
commit cfc4239
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/src/pages/home/sidebar/SidebarScreen/index.js b/src/pages/home/sidebar/SidebarScreen/index.js
@@ -23,18 +23,28 @@ function SidebarScreen(props) {
         }, []),
     );
 
+    /**
+     * Method to hide popover when dragover.
+     */
+    const hidePopoverOnDragOver = useCallback(() => {
+        if (!popoverModal.current) {
+            return;
+        }
+        popoverModal.current.hideCreateMenu();
+    }, []);
+
     /**
      * Method create event listener
      */
     const createDragoverListener = () => {
-        document.addEventListener('dragover', () => popoverModal.current.hideCreateMenu());
+        document.addEventListener('dragover', hidePopoverOnDragOver);
     };
 
     /**
      * Method remove event listener.
      */
     const removeDragoverListener = () => {
-        document.removeEventListener('dragover', () => popoverModal.current.hideCreateMenu());
+        document.removeEventListener('dragover', hidePopoverOnDragOver);
     };
 
     return (