Fix Issue 359:animatedZooms option has a bad interaction with range selector
authorDan Vanderkam <danvk@google.com>
Thu, 16 Aug 2012 21:27:13 +0000 (17:27 -0400)
committerDan Vanderkam <danvk@google.com>
Thu, 16 Aug 2012 21:27:13 +0000 (17:27 -0400)
dygraph.js

index 3fad0d6..cf276ff 100644 (file)
@@ -408,6 +408,12 @@ Dygraph.prototype.__init__ = function(div, file, attrs) {
     // TODO(nikhilk): Add any other stackedGraph checks here.
   }
 
     // 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:
   //
   // Dygraphs has many options, some of which interact with one another.
   // To keep track of everything, we maintain two sets of options:
   //