Fixed enter in modal confirmation dialogs

Fixes #463 Fixes #464 (cherry picked from commit 930742ae2c69a530afe60f76a5824f2722540df8)
2026-04-20 21:54:25 -04:00 · 2020-09-23 00:12:42 +02:00
parent a877c5b25d
commit e4225aa2c2
2 changed files with 7 additions and 5 deletions
@@ -30,10 +30,10 @@ function ConfirmModal(props) {
  useEffect(() => {
    if (isOpen) {
      bindShortcut('enter', onConfirm);
-    } else {
-      unbindShortcut('enter', onConfirm);
+
+      return () => unbindShortcut('enter', onConfirm);
    }
-  }, [onConfirm]);
+  }, [isOpen, onConfirm]);

  return (
    <Modal