Merge pull request Expensify#35402 from mkhutornyi/fix-32743

remove unused popover ref callbacks
kmbcook · Feb 5, 2024 · 89fe932 · 89fe932
2 parents 0ae07be + e63e68c
commit 89fe932
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 8 deletions.
diff --git a/src/components/PopoverProvider/index.tsx b/src/components/PopoverProvider/index.tsx
@@ -27,9 +27,6 @@ function PopoverContextProvider(props: PopoverContextProps) {
         }
 
         activePopoverRef.current.close();
-        if (activePopoverRef.current.onCloseCallback) {
-            activePopoverRef.current.onCloseCallback();
-        }
         activePopoverRef.current = null;
         setIsOpen(false);
     }, []);
@@ -107,9 +104,6 @@ function PopoverContextProvider(props: PopoverContextProps) {
                 closePopover(activePopoverRef.current.anchorRef);
             }
             activePopoverRef.current = popoverParams;
-            if (popoverParams?.onOpenCallback) {
-                popoverParams.onOpenCallback();
-            }
             setIsOpen(true);
         },
         [closePopover],

diff --git a/src/components/PopoverProvider/types.ts b/src/components/PopoverProvider/types.ts
@@ -16,8 +16,6 @@ type AnchorRef = {
     ref: RefObject<View | HTMLDivElement>;
     close: (anchorRef?: RefObject<View | HTMLDivElement>) => void;
     anchorRef: RefObject<View | HTMLDivElement>;
-    onOpenCallback?: () => void;
-    onCloseCallback?: () => void;
 };
 
 export type {PopoverContextProps, PopoverContextValue, AnchorRef};