manually load more entries if needed when pressing a keyboard shortcu…

…t to go to the next entry (#1557)
Athou · Sep 18, 2024 · 191574d · 191574d
1 parent ee7c679
commit 191574d
Showing 1 changed file with 30 additions and 22 deletions.
diff --git a/commafeed-client/src/components/content/FeedEntries.tsx b/commafeed-client/src/components/content/FeedEntries.tsx
@@ -128,28 +128,36 @@ export function FeedEntries() {
     }, [dispatch, entries, viewMode, scrollMarks, scrollingToEntry])
 
     useMousetrap("r", async () => await dispatch(reloadEntries()))
-    useMousetrap(
-        "j",
-        async () =>
-            await dispatch(
-                selectNextEntry({
-                    expand: true,
-                    markAsRead: true,
-                    scrollToEntry: true,
-                })
-            )
-    )
-    useMousetrap(
-        "n",
-        async () =>
-            await dispatch(
-                selectNextEntry({
-                    expand: false,
-                    markAsRead: false,
-                    scrollToEntry: true,
-                })
-            )
-    )
+    useMousetrap("j", async () => {
+        // load more entries if needed
+        // this can happen if the last entry is too large to fit on the screen and the infinite loader doesn't trigger
+        if (hasMore && !loading && selectedEntry === entries[entries.length - 1]) {
+            await dispatch(loadMoreEntries())
+        }
+
+        await dispatch(
+            selectNextEntry({
+                expand: true,
+                markAsRead: true,
+                scrollToEntry: true,
+            })
+        )
+    })
+    useMousetrap("n", async () => {
+        // load more entries if needed
+        // this can happen if the last entry is too large to fit on the screen and the infinite loader doesn't trigger
+        if (hasMore && !loading && selectedEntry === entries[entries.length - 1]) {
+            await dispatch(loadMoreEntries())
+        }
+
+        await dispatch(
+            selectNextEntry({
+                expand: false,
+                markAsRead: false,
+                scrollToEntry: true,
+            })
+        )
+    })
     useMousetrap(
         "k",
         async () =>