fix(dropdown): call event handler when the state change

snowball-tech · Nov 29, 2023 · ebea833 · ebea833
1 parent 217d6ce
commit ebea833
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/packages/fractal/src/components/Dropdown/Dropdown.tsx b/packages/fractal/src/components/Dropdown/Dropdown.tsx
@@ -81,17 +81,18 @@ export const Dropdown = forwardRef<CombinedRefs, DropdownProps>(
  return
  }
 
+ const wasOpened = isOpen
  setIsOpen(isOpened)
 
- if (isFunction(onMenuOpenChange)) {
+ if (isFunction(onMenuOpenChange) && wasOpened !== isOpened) {
  onMenuOpenChange(isOpened)
  }
 
- if (isOpened && isFunction(onOpen)) {
+ if (!wasOpened && isOpened && isFunction(onOpen)) {
  onOpen()
  }
 
- if (!isOpened && isFunction(onClose)) {
+ if (wasOpened && !isOpened && isFunction(onClose)) {
  onClose()
  }
  }