Allows that user specifies only one limit of valueRange and let other one to be calcu...
authormanufitoussi <manu.fitoussi@gmail.com>
Fri, 25 Jan 2013 15:34:32 +0000 (16:34 +0100)
committermanufitoussi <manu.fitoussi@gmail.com>
Fri, 25 Jan 2013 15:34:32 +0000 (16:34 +0100)
auto_tests/tests/range_tests.js
dygraph-options-reference.js
dygraph.js

index 767206f..96e5eb7 100644 (file)
@@ -70,6 +70,14 @@ RangeTestCase.prototype.testRangeSetOperations = function() {
   assertEquals([12, 18], g.xAxisRange());
   assertEquals([10, 40], g.yAxisRange(0));
 
+  g.updateOptions({ valueRange: [10, NaN] });
+  assertEquals([12, 18], g.xAxisRange());
+  assertEquals([10, 44.2], g.yAxisRange(0));
+
+  g.updateOptions({ valueRange: [10, 40] });
+  assertEquals([12, 18], g.xAxisRange());
+  assertEquals([10, 40], g.yAxisRange(0));
+
   g.updateOptions({  });
   assertEquals([12, 18], g.xAxisRange());
   assertEquals([10, 40], g.yAxisRange(0));
index 98cf22f..a6f026f 100644 (file)
@@ -421,7 +421,7 @@ Dygraph.OPTIONS_REFERENCE =  // <JSON>
     "labels": ["Axis display"],
     "type": "Array of two numbers",
     "example": "[10, 110]",
-    "description": "Explicitly set the vertical range of the graph to [low, high]. This may be set on a per-axis basis to define each y-axis separately."
+    "description": "Explicitly set the vertical range of the graph to [low, high]. This may be set on a per-axis basis to define each y-axis separately. You can specify only one limit and let other one (replaced by NaN) to be calculated automatically, by exemple [NaN,30]"
   },
   "labelsDivWidth": {
     "default": "250",
index fc50740..b516f5e 100644 (file)
@@ -2570,7 +2570,10 @@ Dygraph.prototype.computeYAxisRanges_ = function(extremes) {
       axis.computedValueRange = [axis.valueWindow[0], axis.valueWindow[1]];
     } else if (axis.valueRange) {
       // This is a user-set value range for this axis.
-      axis.computedValueRange = [axis.valueRange[0], axis.valueRange[1]];
+      axis.computedValueRange = [
+         !isNaN(axis.valueRange[0]) ? axis.valueRange[0] : axis.extremeRange[0],
+         !isNaN(axis.valueRange[1]) ? axis.valueRange[1] : axis.extremeRange[1]
+      ];
     } else {
       axis.computedValueRange = axis.extremeRange;
     }