From: Dan Vanderkam <danvk@google.com>
Date: Thu, 21 Oct 2010 22:29:14 +0000 (-0400)
Subject: intelligently position labelsDiv in predraw_
X-Git-Tag: v1.0.0~622^2~10
X-Git-Url: https://adrianiainlam.tk/git/?a=commitdiff_plain;h=0abfbd7e563fc0e53015ec875c324890761071ba;hp=880a574fb8382056ec02eb3cb6f4ce1e35463c8f;p=dygraphs.git

intelligently position labelsDiv in predraw_
---

diff --git a/dygraph-canvas.js b/dygraph-canvas.js
index 263cf5c..830ed2a 100644
--- a/dygraph-canvas.js
+++ b/dygraph-canvas.js
@@ -4,7 +4,7 @@
 /**
  * @fileoverview Based on PlotKit, but modified to meet the needs of dygraphs.
  * In particular, support for:
- * - grid overlays 
+ * - grid overlays
  * - error bars
  * - dygraphs attribute system
  */
diff --git a/dygraph.js b/dygraph.js
index a266af0..3d46dc4 100644
--- a/dygraph.js
+++ b/dygraph.js
@@ -655,6 +655,19 @@ Dygraph.prototype.createStatusMessage_ = function() {
 };
 
 /**
+ * Position the labels div so that its right edge is flush with the right edge
+ * of the charting area.
+ */
+Dygraph.prototype.positionLabelsDiv_ = function() {
+  // Don't touch a user-specified labelsDiv.
+  if (this.user_attrs_.hasOwnProperty("labelsDiv")) return;
+
+  var area = this.plotter_.area;
+  var div = this.attr_("labelsDiv");
+  div.style.left = area.x + area.w - this.attr_("labelsDivWidth") + "px";
+};
+
+/**
  * Create the text box to adjust the averaging period
  * @return {Object} The newly-created text box
  * @private
@@ -1587,8 +1600,15 @@ Dygraph.prototype.predraw_ = function() {
                                             this.hidden_, this.layout_,
                                             this.renderOptions_);
 
+  // The roller sits in the bottom left corner of the chart. We don't know where
+  // this will be until the options are available, so it's positioned here.
   this.roller_ = this.createRollInterface_();
 
+  // Same thing applies for the labelsDiv. It's right edge should be flush with
+  // the right edge of the charting area (which may not be the same as the right
+  // edge of the div, if we have two y-axes.
+  this.positionLabelsDiv_();
+
   // If the data or options have changed, then we'd better redraw.
   this.drawGraph_();
 };