From 5e07278e87ac7a5eaa06109ff44639e272945414 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Guillot?= <f@miniflux.net>
Date: Fri, 1 Aug 2025 20:10:44 -0700
Subject: [PATCH] feat(ui): refresh the page when marking as read the last
 visible entry

---
 internal/ui/static/js/app.js | 30 +++++++++++++++++++++++-------
 1 file changed, 23 insertions(+), 7 deletions(-)

diff --git a/internal/ui/static/js/app.js b/internal/ui/static/js/app.js
index 690f3e23..63f6ca5f 100644
--- a/internal/ui/static/js/app.js
+++ b/internal/ui/static/js/app.js
@@ -22,6 +22,15 @@ function getVisibleElements(selector) {
     return [...elements].filter((element) => element.offsetParent !== null);
 }
 
+/**
+ * Get all visible entries on the current page.
+ *
+ * @return {Array<Element>}
+ */
+function getVisibleEntries() {
+    return getVisibleElements(".items .item");
+}
+
 /**
  * Scroll the page to the given element.
  *
@@ -152,7 +161,7 @@ function showKeyboardShortcuts() {
 
 // Mark as read visible items of the current page.
 function markPageAsRead() {
-    const items = getVisibleElements(".items .item");
+    const items = getVisibleEntries();
     const entryIDs = [];
 
     items.forEach((element) => {
@@ -182,19 +191,22 @@ function markPageAsRead() {
 /**
  * Handle entry status changes from the list view and entry view.
  * Focus the next or the previous entry if it exists.
- * @param {string} item Item to focus: "previous" or "next".
- * @param {Element} element
- * @param {boolean} setToRead
+ *
+ * @param {string} navigationDirection Navigation direction: "previous" or "next".
+ * @param {Element} element Element that triggered the action.
+ * @param {boolean} setToRead If true, set the entry to read instead of toggling the status.
+ * @returns {void}
  */
-function handleEntryStatus(item, element, setToRead) {
+function handleEntryStatus(navigationDirection, element, setToRead) {
     const toasting = !element;
     const currentEntry = findEntry(element);
+
     if (currentEntry) {
         if (!setToRead || currentEntry.querySelector(":is(a, button)[data-toggle-status]").dataset.value === "unread") {
             toggleEntryStatus(currentEntry, toasting);
         }
         if (isListView() && currentEntry.classList.contains('current-item')) {
-            switch (item) {
+            switch (navigationDirection) {
             case "previous":
                 goToListItem(-1);
                 break;
@@ -248,6 +260,10 @@ function toggleEntryStatus(element, toasting) {
             element.classList.remove("item-status-" + currentStatus);
             element.classList.add("item-status-" + newStatus);
         }
+
+        if (isListView() && getVisibleEntries().length === 0) {
+            window.location.reload();
+        }
     });
 }
 
@@ -542,7 +558,7 @@ const BOTTOM = -9999;
  * @param {number} offset How many items to jump for focus.
  */
 function goToListItem(offset) {
-    const items = getVisibleElements(".items .item");
+    const items = getVisibleEntries();
     if (items.length === 0) {
         return;
     }