From 8dc6b48ebd73b865c18e261a28f0e97ad304a39b Mon Sep 17 00:00:00 2001 From: Andrew Dolgov Date: Sun, 17 May 2020 22:02:47 +0300 Subject: [PATCH] Headlines.move: when going back to top of active article, use a smarter (?) offset calculation --- js/Headlines.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/js/Headlines.js b/js/Headlines.js index 6b77472e3..ba0f565da 100755 --- a/js/Headlines.js +++ b/js/Headlines.js @@ -844,8 +844,9 @@ define(["dojo/_base/declare"], function (declare) { const row = $("RROW-" + current_id); const ctr = $("headlines-frame"); + const delta_px = Math.max(row.offsetTop, ctr.scrollTop) - Math.min(row.offsetTop, ctr.scrollTop); - if (row && Math.round(row.offsetTop) < Math.round(ctr.scrollTop)) { + if (row && delta_px > 16) { Article.setActive(current_id); Article.cdmMoveToId(current_id, {force: noscroll, event: event}); } else if (prev_id) {