Disable dismiss keyboard shortcut for some views

2 files changed, 22 insertions, 5 deletions

diff --git a/gui/src/renderer/components/KeyboardNavigation.tsx b/gui/src/renderer/components/KeyboardNavigation.tsx
index 4cfba89452..6b3851a49f 100644
--- a/gui/src/renderer/components/KeyboardNavigation.tsx
+++ b/gui/src/renderer/components/KeyboardNavigation.tsx
@@ -1,5 +1,7 @@
 import React, { useCallback, useContext, useEffect, useMemo, useState } from 'react';
+import { useLocation } from 'react-router';
 import { useHistory } from '../lib/history';
+import { disableDismissForRoutes, RoutePath } from '../lib/routes';
 
 interface IKeyboardNavigationProps {
   children: React.ReactElement;
@@ -9,18 +11,22 @@ interface IKeyboardNavigationProps {
 export default function KeyboardNavigation(props: IKeyboardNavigationProps) {
   const history = useHistory();
   const [backAction, setBackAction] = useState<IBackActionConfiguration>();
+  const location = useLocation();
 
   const handleKeyDown = useCallback(
     (event: KeyboardEvent) => {
       if (event.key === 'Escape') {
-        if (event.shiftKey) {
-          history.dismiss(true);
-        } else {
-          backAction?.action();
+        const path = location.pathname as RoutePath;
+        if (!disableDismissForRoutes.includes(path)) {
+          if (event.shiftKey) {
+            history.dismiss(true);
+          } else {
+            backAction?.action();
+          }
         }
       }
     },
-    [history.dismiss, backAction],
+    [history.dismiss, backAction, location.pathname],
   );
 
   useEffect(() => {
diff --git a/gui/src/renderer/lib/routes.ts b/gui/src/renderer/lib/routes.ts
index 22b57960d2..d56c36e82a 100644
--- a/gui/src/renderer/lib/routes.ts
+++ b/gui/src/renderer/lib/routes.ts
@@ -24,6 +24,17 @@ export enum RoutePath {
   filterByProvider = '/select-location/filter-by-provider',
 }
 
+export const disableDismissForRoutes = [
+  RoutePath.launch,
+  RoutePath.login,
+  RoutePath.tooManyDevices,
+  RoutePath.main,
+  RoutePath.redeemVoucher,
+  RoutePath.voucherSuccess,
+  RoutePath.timeAdded,
+  RoutePath.setupFinished,
+];
+
 export function generateRoutePath(
   routePath: RoutePath,
   parameters: Parameters<typeof generatePath>[1],