From 61f740ea7999252898e3c741e2fcb376d4bdcd75 Mon Sep 17 00:00:00 2001 From: Jim Daniels Wasswa Date: Tue, 25 Oct 2022 12:12:49 +0800 Subject: [PATCH] refactor: remove unused event variable and make parameter optional --- packages/components/src/components/modal/modal.tsx | 2 +- packages/components/src/hooks/use-onclickoutside.ts | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/components/src/components/modal/modal.tsx b/packages/components/src/components/modal/modal.tsx index 05385f5413a7..1aee5dac3ad5 100644 --- a/packages/components/src/components/modal/modal.tsx +++ b/packages/components/src/components/modal/modal.tsx @@ -89,7 +89,7 @@ const ModalElement = ({ ); }; - const closeModal = (e: IClickEvent) => { + const closeModal = () => { if (is_open) toggleModal?.(); }; diff --git a/packages/components/src/hooks/use-onclickoutside.ts b/packages/components/src/hooks/use-onclickoutside.ts index 6a8125d8ebee..8cf8d6726c6c 100644 --- a/packages/components/src/hooks/use-onclickoutside.ts +++ b/packages/components/src/hooks/use-onclickoutside.ts @@ -6,7 +6,7 @@ export interface IClickEvent extends MouseEvent { export const useOnClickOutside = ( ref: RefObject, - handler: (event: IClickEvent) => void, + handler: (event?: IClickEvent) => void, validationFn: (event: IClickEvent) => boolean ) => { React.useEffect(() => {