moved default parameter (ToolJet#1041)

VilijaSan · Oct 16, 2021 · 02434ce · 02434ce
1 parent 78d0ce3
commit 02434ce
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/frontend/src/_hooks/use-popover.jsx b/frontend/src/_hooks/use-popover.jsx
@@ -13,7 +13,7 @@ const useEscapeHandler = (handler = noop, dependencies = []) => {
       document === null || document === void 0 ? void 0 : document.removeEventListener('keyup', escapeHandler);
   }, dependencies);
 };
-const useClickOutside = (handler = noop, dependencies) => {
+const useClickOutside = (dependencies, handler = noop) => {
   const callbackRef = useRef(handler);
   const ref = useRef(null);
   const outsideClickHandler = (e) => {
@@ -43,7 +43,7 @@ const usePopover = (defaultOpen = false) => {
   const toggle = useCallback(() => setOpen(!open), []);
   const close = useCallback(() => setOpen(false), []);
   useEscapeHandler(close, []);
-  const contentRef = useClickOutside(open ? close : undefined, []);
+  const contentRef = useClickOutside([], open ? close : undefined);
   const trigger = {
     ref: triggerRef,
     onClick: toggle,