Fixed enter in modal confirmation dialogs

(cherry picked from commit 930742ae2c69a530afe60f76a5824f2722540df8)
pull/1689/head
Taloth Saldono 4 years ago committed by Qstick
parent 58cb0dc556
commit ace7371b69

@ -30,10 +30,10 @@ function ConfirmModal(props) {
useEffect(() => { useEffect(() => {
if (isOpen) { if (isOpen) {
bindShortcut('enter', onConfirm); bindShortcut('enter', onConfirm);
} else {
unbindShortcut('enter', onConfirm); return () => unbindShortcut('enter', onConfirm);
} }
}, [onConfirm]); }, [isOpen, onConfirm]);
return ( return (
<Modal <Modal

@ -56,9 +56,11 @@ function keyboardShortcuts(WrappedComponent) {
} }
unbindShortcut = (key) => { unbindShortcut = (key) => {
if (this._mousetrap != null) {
delete this._mousetrapBindings[key]; delete this._mousetrapBindings[key];
this._mousetrap.unbind(key); this._mousetrap.unbind(key);
} }
}
unbindAllShortcuts = () => { unbindAllShortcuts = () => {
const keys = Object.keys(this._mousetrapBindings); const keys = Object.keys(this._mousetrapBindings);

Loading…
Cancel
Save