Fixed: Enter on Delete profile confirmation deleting all unused profiles (#4241)

5 years ago · ed72713ba7
parent 2fbcf17c8d
commit ed72713ba7
1 changed files with 9 additions and 3 deletions
--- a/frontend/src/Components/Modal/ConfirmModal.js
+++ b/frontend/src/Components/Modal/ConfirmModal.js
@ -23,11 +23,16 @@ function ConfirmModal(props) {
    isSpinning,
    onConfirm,
    onCancel,
-    bindShortcut
+    bindShortcut,
+    unbindShortcut
  } = props;

  useEffect(() => {
-    bindShortcut('enter', onConfirm);
+    if (isOpen) {
+      bindShortcut('enter', onConfirm);
+    } else {
+      unbindShortcut('enter', onConfirm);
+    }
  }, [onConfirm]);

  return (
@ -81,7 +86,8 @@ ConfirmModal.propTypes = {
  isSpinning: PropTypes.bool.isRequired,
  onConfirm: PropTypes.func.isRequired,
  onCancel: PropTypes.func.isRequired,
-  bindShortcut: PropTypes.func.isRequired
+  bindShortcut: PropTypes.func.isRequired,
+  unbindShortcut: PropTypes.func.isRequired
 };

 ConfirmModal.defaultProps = {