Merge pull request #2718 from omnivore-app/fix/web-cleanups

Some cleanups based on reddit feedback

packages/web/components/templates/KeyboardShortcutListModal.tsx

@@ -76,12 +76,6 @@ const readerCommands = () => {
       shortcutKeyDescription: 'f',
       callback: () => {},
     },
-    {
-      shortcutKeys: ['escape'],
-      actionDescription: 'End multi select',
-      shortcutKeyDescription: 'Escape',
-      callback: () => {},
-    },
     {
       actionDescription: 'Return to library',
       shortcutKeys: ['u'],

packages/web/lib/keyboardShortcuts/navigationShortcuts.ts

@@ -15,7 +15,9 @@ export function navigationCommands(
       shortcutKeys: ['b'],
       actionDescription: 'Go back',
       shortcutKeyDescription: 'b',
-      callback: () => router?.back(),
+      callback: () => {
+        router?.back()
+      },
     },
   ]
 }

packages/web/pages/[username]/[slug]/index.tsx

@@ -79,8 +79,6 @@ export default function Home(): JSX.Element {
     })
   }, [articleData?.article.article])
 
-  useKeyboardShortcuts(navigationCommands(router))
-
   const actionHandler = useCallback(
     async (action: string, arg?: unknown) => {
       switch (action) {

packages/web/pages/settings/feeds/index.tsx

@@ -90,7 +90,7 @@ export default function Rss(): JSX.Element {
   return (
     <SettingsTable
       pageId={'feeds'}
-      pageInfoLink="https://docs.omnivore.app/settings/feeds.html"
+      pageInfoLink="https://docs.omnivore.app/using/feeds.html"
       headerTitle="Subscribed feeds"
       createTitle="Add feed"
       createAction={() => {