From dfc38e3413d840206dc33d8ae91863f97aff5134 Mon Sep 17 00:00:00 2001
From: Rory Abraham <rory@expensify.com>
Date: Wed, 6 Jul 2022 16:54:37 -0700
Subject: [PATCH 1/2] Make it possible to locally prevent event propagation

---
 src/libs/KeyboardShortcut/index.js           | 5 +++--
 src/pages/signin/ChangeExpensifyLoginLink.js | 4 ++++
 src/pages/signin/PasswordForm.js             | 1 +
 3 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/libs/KeyboardShortcut/index.js b/src/libs/KeyboardShortcut/index.js
index f948d69914610..0f2863ddda31d 100644
--- a/src/libs/KeyboardShortcut/index.js
+++ b/src/libs/KeyboardShortcut/index.js
@@ -103,8 +103,9 @@ function bindHandlerToKeydownEvent(event) {
 }
 
 // Make sure we don't add multiple listeners
-document.removeEventListener('keydown', bindHandlerToKeydownEvent, {capture: true});
-document.addEventListener('keydown', bindHandlerToKeydownEvent, {capture: true});
+const rootNode = document.getElementById('root');
+rootNode.removeEventListener('keydown', bindHandlerToKeydownEvent);
+rootNode.addEventListener('keydown', bindHandlerToKeydownEvent);
 
 /**
  * Unsubscribes a keyboard event handler.
diff --git a/src/pages/signin/ChangeExpensifyLoginLink.js b/src/pages/signin/ChangeExpensifyLoginLink.js
index 8d48d708bc9e2..fdf7dc437c2d3 100755
--- a/src/pages/signin/ChangeExpensifyLoginLink.js
+++ b/src/pages/signin/ChangeExpensifyLoginLink.js
@@ -40,6 +40,10 @@ const ChangeExpensifyLoginLink = props => (
         <TouchableOpacity
             style={[styles.link]}
             onPress={Session.clearSignInData}
+            onKeyDownCapture={(e) => {
+                e.stopPropagation();
+                Session.clearSignInData();
+            }}
             underlayColor={themeColors.componentBG}
         >
             <Text style={[styles.link]}>
diff --git a/src/pages/signin/PasswordForm.js b/src/pages/signin/PasswordForm.js
index 0801487c88038..571f2353bf6be 100755
--- a/src/pages/signin/PasswordForm.js
+++ b/src/pages/signin/PasswordForm.js
@@ -181,6 +181,7 @@ class PasswordForm extends React.Component {
                 <View>
                     <Button
                         success
+                        pressOnEnter
                         style={[styles.mv3]}
                         text={this.props.translate('common.signIn')}
                         isLoading={this.props.account.loading}

From ee60e744a7d5554ffdb6b45937361ab44c1bd7c6 Mon Sep 17 00:00:00 2001
From: Rory Abraham <rory@expensify.com>
Date: Tue, 12 Jul 2022 09:58:51 -0700
Subject: [PATCH 2/2] Add window event listeners too

---
 src/libs/KeyboardShortcut/index.js | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/libs/KeyboardShortcut/index.js b/src/libs/KeyboardShortcut/index.js
index 0f2863ddda31d..39ffed7c65408 100644
--- a/src/libs/KeyboardShortcut/index.js
+++ b/src/libs/KeyboardShortcut/index.js
@@ -106,6 +106,8 @@ function bindHandlerToKeydownEvent(event) {
 const rootNode = document.getElementById('root');
 rootNode.removeEventListener('keydown', bindHandlerToKeydownEvent);
 rootNode.addEventListener('keydown', bindHandlerToKeydownEvent);
+window.removeEventListener('keydown', bindHandlerToKeydownEvent);
+window.addEventListener('keydown', bindHandlerToKeydownEvent);
 
 /**
  * Unsubscribes a keyboard event handler.