From 0e4f67bf2b4cd2b387aa6b5da56f57b9c59ceadd Mon Sep 17 00:00:00 2001 From: Andrew Dolgov Date: Tue, 10 Dec 2019 13:03:49 +0300 Subject: [PATCH] with previous change, we don't actually need to automatically track active articles now at all in combined mode. --- js/Headlines.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/js/Headlines.js b/js/Headlines.js index 25244f195..6a3296205 100755 --- a/js/Headlines.js +++ b/js/Headlines.js @@ -272,7 +272,7 @@ define(["dojo/_base/declare"], function (declare) { try { Headlines.unpackVisible(); - if (App.isCombinedMode()) { + /*if (App.isCombinedMode()) { Headlines.updateFloatingTitle(); const ctr = $("headlines-frame"); @@ -289,13 +289,13 @@ define(["dojo/_base/declare"], function (declare) { /*console.log(row.getAttribute("data-article-title"), row.offsetTop, row.offsetHeight, ctr.scrollTop, ctr.offsetHeight, this.isChildVisible(row, ctr));*/ - if (this.isChildVisible(row, ctr)) { + /* if (this.isChildVisible(row, ctr)) { Article.setActive(row.getAttribute("data-article-id")); break; } } } - } + } */ if (!Feeds.infscroll_disabled && !Feeds.infscroll_in_progress) { const hsp = $("headlines-spacer");