fix some minor issues with onOpenChange events with useDisclosure

Southclaws · Jan 1, 2024 · 489b882 · 489b882 · vercel · Jan 1, 2024
1 parent 34cd940
commit 489b882
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/web/src/utils/useDisclosure.ts b/web/src/utils/useDisclosure.ts
@@ -36,11 +36,13 @@ export function useDisclosure(props: UseDisclosureProps = {}) {
     onClose: onCloseProp,
     onOpen: onOpenProp,
     isOpen: isOpenProp,
+    onOpenChange: onOpenChangeProp,
     id: idProp,
   } = props;
 
   const handleOpen = useCallbackRef(onOpenProp);
   const handleClose = useCallbackRef(onCloseProp);
+  const handleOpenChange = useCallbackRef(onOpenChangeProp);
 
   const [isOpenState, setIsOpen] = useState(props.defaultIsOpen || false);
 
@@ -76,12 +78,13 @@ export function useDisclosure(props: UseDisclosureProps = {}) {
   const onOpenChange = useCallback(
     (event: OpenChangeEvent) => {
       if (event.open) {
-        onClose();
-      } else {
         onOpen();
+      } else {
+        onClose();
       }
+      handleOpenChange?.(event);
     },
-    [onOpen, onClose],
+    [onOpen, onClose, handleOpenChange],
   );
 
   function getButtonProps(props: HTMLProps = {}): HTMLProps {