From 283f1b3002172ab52b9894520f4981f045497734 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=CE=BCherman?= <143259389+uherman@users.noreply.github.com>
Date: Fri, 22 Sep 2023 20:18:57 +0200
Subject: [PATCH] fix: check activateClickOutside on outside click (#1082)

handleClickOutside checks the state of activateClickOutside when called.
This allows setting activateClickOutside when the component is mounted.
---
 src/lib/drawer/Drawer.svelte | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/lib/drawer/Drawer.svelte b/src/lib/drawer/Drawer.svelte
index 3784ad2e6..090511f3b 100644
--- a/src/lib/drawer/Drawer.svelte
+++ b/src/lib/drawer/Drawer.svelte
@@ -45,6 +45,8 @@
     hidden = !hidden;
   };
 
+  const handleClickOutside = () => activateClickOutside && !hidden && handleDrawer();
+
   let backdropDivClass = twMerge('fixed top-0 left-0 z-50 w-full h-full', backdrop && bgColor, backdrop && bgOpacity);
 
   function clickOutsideWrapper(node: HTMLElement, callback: () => void) {
@@ -59,7 +61,7 @@
     <div role="presentation" class={backdropDivClass} />
   {/if}
 
-  <div use:clickOutsideWrapper={() => !hidden && handleDrawer()} {id} {...$$restProps} class={twMerge(divClass, width, position, placements[placement], $$props.class)} transition:multiple={transitionParams} tabindex="-1" aria-controls={id} aria-labelledby={id}>
+  <div use:clickOutsideWrapper={handleClickOutside} {id} {...$$restProps} class={twMerge(divClass, width, position, placements[placement], $$props.class)} transition:multiple={transitionParams} tabindex="-1" aria-controls={id} aria-labelledby={id}>
     <slot {hidden} />
   </div>
 {/if}