diff --git a/js/App.js b/js/App.js index 8776aab81..55844873c 100644 --- a/js/App.js +++ b/js/App.js @@ -98,7 +98,16 @@ const App = { return elem.offsetTop + elem.offsetHeight <= ctr.scrollTop + ctr.offsetHeight && elem.offsetTop >= ctr.scrollTop; - } + }, + scrollTo: function (elem, ctr, params = {}) { + const force_to_top = params.force_to_top || false; + + if (!elem || !ctr) return; + + if (force_to_top || !App.Scrollable.fitsInContainer(elem, ctr)) { + ctr.scrollTop = elem.offsetTop; + } + } }, dialogOf: function (elem) { @@ -541,9 +550,7 @@ const App = { this.initSecondStage(); }, Error: { - fatal: function (error, params) { - params = params || {}; - + fatal: function (error, params = {}) { if (params.code) { if (params.code == 6) { window.location.href = "index.php"; @@ -557,9 +564,7 @@ const App = { return this.report(error, Object.extend({title: __("Fatal error")}, params)); }, - report: function(error, params) { - params = params || {}; - + report: function(error, params = {}) { if (!error) return; console.error("[Error.report]", error, params); diff --git a/js/Article.js b/js/Article.js index a42d3af67..81917fd47 100644 --- a/js/Article.js +++ b/js/Article.js @@ -383,9 +383,7 @@ const Article = { dialog.show(); }, - cdmMoveToId: function (id, params) { - params = params || {}; - + cdmMoveToId: function (id, params = {}) { const force_to_top = params.force_to_top || false; const ctr = $("headlines-frame"); diff --git a/js/Headlines.js b/js/Headlines.js index dadf79c1a..16026f593 100755 --- a/js/Headlines.js +++ b/js/Headlines.js @@ -814,9 +814,7 @@ const Headlines = { Feeds.reloadCurrent(); }, - selectionToggleUnread: function (params) { - params = params || {}; - + selectionToggleUnread: function (params = {}) { const cmode = params.cmode != undefined ? params.cmode : 2; const no_error = params.no_error || false; const ids = params.ids || Headlines.getSelected(); @@ -882,9 +880,7 @@ const Headlines = { if (row) row.toggleClassName("published"); }, - move: function (mode, params) { - params = params || {}; - + move: function (mode, params = {}) { const no_expand = params.no_expand || false; const force_previous = params.force_previous || this.default_force_previous; const force_to_top = params.force_to_top || this.default_force_to_top; @@ -930,6 +926,17 @@ const Headlines = { } else { Article.view(next_id, no_expand); } + } else if (App.isCombinedMode()) { + // try to show hsp if no next article exists, in case there's useful information like first_id_changed etc + const row = $("RROW-" + current_id); + + if (row) { + const sibling = row.nextSibling; + + if (sibling && Element.visible(sibling) && sibling.id == "headlines-spacer") { + App.Scrollable.scrollTo(sibling, $("headlines-frame")); + } + } } } else if (mode === "prev") { if (prev_id || current_id) {