From 5819e4007f12a982a6c46c2fb82625db7bf6fedf Mon Sep 17 00:00:00 2001
From: "Cheng-Han, Wu" <jackymaxj@gmail.com>
Date: Sat, 28 May 2016 00:31:43 +0800
Subject: [PATCH] Fix when scroll animate duration greater than debounce will
 cause scrolling flicking

---
 public/js/syncscroll.js | 37 +++++++++++++++++++------------------
 1 file changed, 19 insertions(+), 18 deletions(-)

diff --git a/public/js/syncscroll.js b/public/js/syncscroll.js
index 53070fc..c8b45e7 100644
--- a/public/js/syncscroll.js
+++ b/public/js/syncscroll.js
@@ -115,10 +115,7 @@ var viewScrollThrottle = 10;
 var buildMapThrottle = 100;
 
 var viewScrolling = false;
-var viewScrollingDebounce = 200;
-
 var editScrolling = false;
-var editScrollingDebounce = 200;
 
 ui.area.codemirrorScroll.on('scroll', _.throttle(syncScrollToView, editScrollThrottle));
 ui.area.view.on('scroll', _.throttle(syncScrollToEdit, viewScrollThrottle));
@@ -214,12 +211,8 @@ function buildMapInner(callback) {
     if (loaded && callback) callback();
 }
 
-// sync view scroll progress to edit 
-var viewScrollingTimeout = _.debounce(viewScrollingTimeoutInner, viewScrollingDebounce);
-
-function viewScrollingTimeoutInner() {
-    viewScrolling = false;
-}
+// sync view scroll progress to edit
+var viewScrollingTimer = null;
 
 function syncScrollToEdit(e) {
     if (currentMode != modeType.both || !syncscroll) return;
@@ -280,20 +273,22 @@ function syncScrollToEdit(e) {
     
     var posDiff = Math.abs(scrollInfo.top - posTo);
     var duration = posDiff / 50;
+    duration = duration >= 100 ? duration : 100;
     ui.area.codemirrorScroll.stop(true, true).animate({
         scrollTop: posTo
-    }, duration >= 100 ? duration : 100, "linear");
+    }, duration, "linear");
     
     viewScrolling = true;
-    viewScrollingTimeout();
+    clearTimeout(viewScrollingTimer);
+    viewScrollingTimer = setTimeout(viewScrollingTimeoutInner, duration * 1.2);
+}
+
+function viewScrollingTimeoutInner() {
+    viewScrolling = false;
 }
 
 // sync edit scroll progress to view
-var editScrollingTimeout = _.debounce(editScrollingTimeoutInner, editScrollingDebounce);
-
-function editScrollingTimeoutInner() {
-    editScrolling = false;
-}
+var editScrollingTimer = null;
 
 function syncScrollToView(event, _lineNo) {
     if (currentMode != modeType.both || !syncscroll) return;
@@ -336,10 +331,16 @@ function syncScrollToView(event, _lineNo) {
     
     var posDiff = Math.abs(ui.area.view.scrollTop() - posTo);
     var duration = posDiff / 50;
+    duration = duration >= 100 ? duration : 100;
     ui.area.view.stop(true, true).animate({
         scrollTop: posTo
-    }, duration >= 100 ? duration : 100, "linear");
+    }, duration, "linear");
     
     editScrolling = true;
-    editScrollingTimeout();
+    clearTimeout(editScrollingTimer);
+    editScrollingTimer = setTimeout(editScrollingTimeoutInner, duration * 1.2);
+}
+
+function editScrollingTimeoutInner() {
+    editScrolling = false;
 }
\ No newline at end of file