From d1754390cc1c45eb469daf0221e235ca1a02d555 Mon Sep 17 00:00:00 2001 From: Veit Lehmann Date: Tue, 25 Jun 2013 06:08:44 +0300 Subject: [PATCH] only do article update operations once when scrolling I discovered that DOM operations and function calls were done several times inside the 100px mirror when scrolling through articles. Especially painful on seamless scrolling systems like Mac OS, where dozens of updates would slow down this part dramatically. This change fixes it. --- js/viewfeed.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/js/viewfeed.js b/js/viewfeed.js index 9c48fbcdb..1c5811fe7 100644 --- a/js/viewfeed.js +++ b/js/viewfeed.js @@ -1281,7 +1281,8 @@ function headlines_scroll_handler(e) { var child = rows[i]; if ($("headlines-frame").scrollTop < child.offsetTop && - child.offsetTop - $("headlines-frame").scrollTop < 100) { + child.offsetTop - $("headlines-frame").scrollTop < 100 && + child.id.replace("RROW-", "") != _active_article_id) { if (_active_article_id) { var row = $("RROW-" + _active_article_id);