Fix keyboard navigation with scroll_to_article_header=0

fossar · Nov 24, 2023 · b4d0434 · b4d0434
1 parent e0c9805
commit b4d0434
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/client/js/templates/EntriesPage.jsx b/client/js/templates/EntriesPage.jsx
@@ -587,19 +587,23 @@ export default class StateHolder extends React.Component {
      * @param {number} id of entry
      */
     activateEntry(id) {
+        const entry = this.state.entries.find((entry) => id === entry.id);
+        const element = document.querySelector(`.entry[data-entry-id="${id}"]`);
+
         if (this.props.configuration.autoCollapse) {
             this.collapseAllEntries();
         }
 
         this.setSelectedEntry(id);
 
+        // scroll to entry
+        element.scrollIntoView();
+
         // show/hide (with toolbar)
         this.setEntryExpanded(id, true);
 
         // automark as read
-        const entry = this.state.entries.find((entry) => id === entry.id);
-        const autoMarkAsRead = selfoss.isAllowedToWrite() && this.props.configuration.autoMarkAsRead && entry.unread == 1;
-        if (autoMarkAsRead) {
+        if (selfoss.isAllowedToWrite() && this.props.configuration.autoMarkAsRead && entry.unread == 1) {
             this.markEntryRead(id, true);
         }
     }