From: Dan Vanderkam <danvk@google.com>
Date: Thu, 16 Aug 2012 21:27:13 +0000 (-0400)
Subject: Fix Issue 359:animatedZooms option has a bad interaction with range selector
X-Git-Tag: v1.0.0~187
X-Git-Url: https://adrianiainlam.tk/git/?a=commitdiff_plain;h=e71332d66382629c5014aa75d7c5f23b8686d9ff;p=dygraphs.git

Fix Issue 359:animatedZooms option has a bad interaction with range selector
---

diff --git a/dygraph.js b/dygraph.js
index 3fad0d6..cf276ff 100644
--- a/dygraph.js
+++ b/dygraph.js
@@ -408,6 +408,12 @@ Dygraph.prototype.__init__ = function(div, file, attrs) {
     // TODO(nikhilk): Add any other stackedGraph checks here.
   }
 
+  // These two options have a bad interaction. See issue 359.
+  if (attrs.showRangeSelector && attrs.animatedZooms) {
+    this.warn('You should not set animatedZooms=true when using the range selector.');
+    attrs.animatedZooms = false;
+  }
+
   // Dygraphs has many options, some of which interact with one another.
   // To keep track of everything, we maintain two sets of options:
   //