Fixed: Enter on Delete profile confirmation deleting all unused profiles

pull/3660/head
Mark McDowall 5 years ago
parent 72afb28c30
commit 4ddf4a22a3

@ -23,11 +23,16 @@ function ConfirmModal(props) {
isSpinning, isSpinning,
onConfirm, onConfirm,
onCancel, onCancel,
bindShortcut bindShortcut,
unbindShortcut
} = props; } = props;
useEffect(() => { useEffect(() => {
bindShortcut('enter', onConfirm); if (isOpen) {
bindShortcut('enter', onConfirm);
} else {
unbindShortcut('enter', onConfirm);
}
}, [onConfirm]); }, [onConfirm]);
return ( return (
@ -81,7 +86,8 @@ ConfirmModal.propTypes = {
isSpinning: PropTypes.bool.isRequired, isSpinning: PropTypes.bool.isRequired,
onConfirm: PropTypes.func.isRequired, onConfirm: PropTypes.func.isRequired,
onCancel: PropTypes.func.isRequired, onCancel: PropTypes.func.isRequired,
bindShortcut: PropTypes.func.isRequired bindShortcut: PropTypes.func.isRequired,
unbindShortcut: PropTypes.func.isRequired
}; };
ConfirmModal.defaultProps = { ConfirmModal.defaultProps = {

Loading…
Cancel
Save