Check for new options before updating synchronised graphs (Fixes #760)
authorJustin Standring <me@justinstandring.com>
Fri, 8 Jul 2016 23:05:40 +0000 (16:05 -0700)
committerJustin Standring <me@justinstandring.com>
Mon, 11 Jul 2016 21:43:03 +0000 (14:43 -0700)
src/extras/synchronizer.js [changed mode: 0644->0755]

old mode 100644 (file)
new mode 100755 (executable)
index bf5b6f3..6f369f3
@@ -153,6 +153,16 @@ var synchronize = function(/* dygraphs..., opts */) {
   };
 };
 
+function arraysAreEqual(a, b) {
+  if (!Array.isArray(a) || !Array.isArray(b)) return false;
+  var i = a.length;
+  if (i !== b.length) return false;
+  while (i--) {
+    if (a[i] !== b[i]) return false;
+  }
+  return true;
+}
+
 function attachZoomHandlers(gs, syncOpts, prevCallbacks) {
   var block = false;
   for (var i = 0; i < gs.length; i++) {
@@ -173,6 +183,13 @@ function attachZoomHandlers(gs, syncOpts, prevCallbacks) {
             }
             continue;
           }
+
+          // Only redraw if there are new options
+          if (arraysAreEqual(opts.dateWindow, gs[j].getOption('dateWindow')) && 
+              arraysAreEqual(opts.valueRange, gs[j].getOption('valueRange'))) {
+            continue;
+          }
+
           gs[j].updateOptions(opts);
         }
         block = false;