Headlines.move: when going back to top of active article, use a smarter (?) offset calculation

5 years ago · 8dc6b48ebd
parent 06d2c65193
commit 8dc6b48ebd
1 changed files with 2 additions and 1 deletions
--- 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) {