only reserve space for y2 axis when it will be drawn
authorDan Vanderkam <danvk@google.com>
Tue, 30 Apr 2013 16:09:11 +0000 (12:09 -0400)
committerDan Vanderkam <danvk@google.com>
Tue, 30 Apr 2013 16:09:11 +0000 (12:09 -0400)
plugins/axes.js
tests/two-axes.html

index f7865c8..cacda86 100644 (file)
@@ -71,9 +71,12 @@ axes.prototype.layout = function(e) {
   }
 
   if (g.numAxes() == 2) {
-    // TODO(danvk): per-axis setting.
-    var w = g.getOption('yAxisLabelWidth') + 2 * g.getOption('axisTickSize');
-    e.reserveSpaceRight(w);
+    // TODO(danvk): introduce a 'drawAxis' per-axis property.
+    if (g.getOption('drawYAxis')) {
+      // TODO(danvk): per-axis setting.
+      var w = g.getOption('yAxisLabelWidth') + 2 * g.getOption('axisTickSize');
+      e.reserveSpaceRight(w);
+    }
   } else if (g.numAxes() > 2) {
     g.error("Only two y-axes are supported at this time. (Trying " +
             "to use " + g.numAxes() + ")");
index 3f40b07..cfc36a5 100644 (file)
@@ -67,7 +67,7 @@
             labels: [ 'Date', 'Y1', 'Y2', 'Y3', 'Y4' ],
             labelsKMB: true,
             ylabel: 'Primary y-axis',
-            y2label: 'Secondary y-axis',
+            y2label: 'Secondary y-axis'
           }
       );