fix: check onKeyUp instead of onKeyDown

This commit is contained in:
Baptiste Grob
2021-01-27 15:32:26 +01:00
parent 5b7777bb98
commit 1576da01c4

View File

@@ -40,6 +40,7 @@ class ChallengeModalCtrl extends PureViewCtrl<unknown, ChallengeModalState> {
application!: WebApplication;
challenge!: Challenge;
onDismiss!: () => void;
submitting = false;
/** @template */
protectionsSessionDurations = ProtectionSessionDurations;
@@ -177,6 +178,10 @@ class ChallengeModalCtrl extends PureViewCtrl<unknown, ChallengeModalState> {
if (!this.validate()) {
return;
}
if (this.submitting) {
return;
}
this.submitting = true;
await this.setState({ processing: true });
const values: ChallengeValue[] = [];
for (const inputValue of Object.values(this.getState().values)) {
@@ -200,6 +205,7 @@ class ChallengeModalCtrl extends PureViewCtrl<unknown, ChallengeModalState> {
} else {
this.setState({ processing: false });
}
this.submitting = false;
}, 50);
}
@@ -378,10 +384,9 @@ function ChallengePrompts({
ctrl.state.values[prompt.id]!.value = value;
ctrl.onTextValueChange(prompt);
}}
onKeyDown={(event) => {
onKeyUp={(event) => {
if (event.key === 'Enter') {
event.preventDefault();
event.stopPropagation();
ctrl.submit();
}
}}