fix(modal): prevent closing from being executed twice

Author: Gregor Woiwode

packages/kit-headless/src/components/modal/modal.tsx

@@ -66,8 +66,6 @@ export const Modal = component$((props: ModalProps) => {
  // fixes modal scrollbar flickering
  modal.style.left = `${scrollbarWidth}px`;
 
- closing(modal, props.onClose$);
-
  showSig.value = false;
  }
  };
@@ -82,10 +80,10 @@ export const Modal = component$((props: ModalProps) => {
  lockScroll();
  } else {
  unlockScroll(scrollbarWidth);
- closing(modal, props.onClose$);
  }
 
  cleanup(() => {
+ closing(modal, props.onClose$);
  deactivateFocusTrap(focusTrap);
 
  // prevents closing animation scrollbar flickers (chrome & edge)