no more renderer options!
authorDan Vanderkam <dan@dygraphs.com>
Mon, 9 May 2011 16:04:22 +0000 (09:04 -0700)
committerDan Vanderkam <dan@dygraphs.com>
Mon, 9 May 2011 16:04:22 +0000 (09:04 -0700)
dygraph-canvas.js
dygraph.js

index 9ac88be..3aba3cd 100644 (file)
@@ -433,6 +433,14 @@ DygraphCanvasRenderer.isSupported = function(canvasName) {
 };
 
 /**
+ * @param { [String] } colors Array of color strings. Should have one entry for
+ * each series to be rendered.
+ */
+DygraphCanvasRenderer.prototype.setColors = function(colors) {
+  this.colorScheme_ = colors;
+};
+
+/**
  * Draw an X/Y grid on top of the existing plot
  */
 DygraphCanvasRenderer.prototype.render = function() {
@@ -837,7 +845,6 @@ DygraphCanvasRenderer.prototype._renderAnnotations = function() {
 DygraphCanvasRenderer.prototype._renderLineChart = function() {
   // TODO(danvk): use this.attr_ for many of these.
   var context = this.elementContext;
-  var colorScheme = this.attr_('colors');
   var fillAlpha = this.attr_('fillAlpha');
   var errorBars = this.attr_("errorBars");
   var fillGraph = this.attr_("fillGraph");
@@ -855,7 +862,7 @@ DygraphCanvasRenderer.prototype._renderLineChart = function() {
   // TODO(danvk): Move this mapping into Dygraph and get it out of here.
   this.colors = {}
   for (var i = 0; i < setCount; i++) {
-    this.colors[setNames[i]] = colorScheme[i % colorScheme.length];
+    this.colors[setNames[i]] = this.colorScheme_[i % this.colorScheme_.length];
   }
 
   // Update Points
index ac1bc03..aeb30e9 100644 (file)
@@ -896,6 +896,8 @@ Dygraph.prototype.setColors_ = function() {
       this.colors_.push(colorStr);
     }
   }
+
+  this.plotter_.setColors(this.colors_);
 };
 
 /**