add new onBeforeOpen prop on Tooltip to avoid overwriting of onOpen prop

Author: pettinarip

src/components/Glossary/GlossaryTooltip/index.tsx

@@ -26,7 +26,7 @@ const GlossaryTooltip = ({ children, termKey }: GlossaryTooltipProps) => {
             options={{ ns: "glossary-tooltip" }}
           />
         }
-        onOpen={() => {
+        onBeforeOpen={() => {
           trackCustomEvent({
             eventCategory: "Glossary Tooltip",
             eventAction: cleanPath(asPath),

src/components/Tooltip/index.tsx

@@ -14,9 +14,15 @@ import { isMobile } from "@/lib/utils/isMobile"
 export interface IProps extends PopoverProps {
   content: ReactNode
   children?: ReactNode
+  onBeforeOpen?: () => void
 }
 
-const Tooltip: React.FC<IProps> = ({ content, children, ...rest }) => {
+const Tooltip: React.FC<IProps> = ({
+  content,
+  children,
+  onBeforeOpen,
+  ...rest
+}) => {
   const { isOpen, onOpen, onClose } = useDisclosure()
 
   // Close the popover when the user scrolls.
@@ -46,19 +52,19 @@ const Tooltip: React.FC<IProps> = ({ content, children, ...rest }) => {
   }, [isOpen, onClose])
 
   const handleOpen = () => {
+    onBeforeOpen?.()
     onOpen()
-    rest.onOpen?.()
   }
 
   return (
     <Popover
+      isOpen={isOpen}
+      onOpen={handleOpen}
+      onClose={onClose}
       placement="top"
       trigger={isMobile() ? "click" : "hover"}
       gutter={8}
       {...rest}
-      isOpen={isOpen}
-      onOpen={handleOpen}
-      onClose={onClose}
     >
       <PopoverTrigger>{children}</PopoverTrigger>
       <PopoverContent>