fix: redirect after esc button in any setting page (#193)

* Fix the redirect after exc button in any setting page * handle esc only on opened modals * code styling change Co-authored-by: nina992 <nouralali992@gmail.com>
reearth · Apr 13, 2022 · c8ec354 · c8ec354
1 parent a90dda4
commit c8ec354
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/components/atoms/Modal/ModalFrame/index.tsx b/src/components/atoms/Modal/ModalFrame/index.tsx
@@ -21,7 +21,9 @@ const Modal: React.FC<Props> = ({ size, isVisible, onClose, children }) => {
     unmountOnExit: true,
   });
 
-  const handleClose = useCallback(() => onClose?.(), [onClose]);
+  const handleClose = useCallback(() => {
+    if (isVisible) onClose?.();
+  }, [onClose, isVisible]);
 
   useKeyPressEvent("Escape", handleClose);