fix: check onKeyUp instead of onKeyDown

standardnotes · Jan 27, 2021 · 1576da0 · 1576da0
1 parent 5b7777b
commit 1576da0
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/app/assets/javascripts/views/challenge_modal/challenge_modal.tsx b/app/assets/javascripts/views/challenge_modal/challenge_modal.tsx
@@ -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();
               }
             }}