Bug: Allow points where canvas-y === 0

The check for `utils.isOK` blacklists the value 0. In the context of
validating selected points and reporting them in the legend, any point
where canvas-y zero should be included.

diff --git a/src/dygraph.js b/src/dygraph.js
index bc75fef..a5e326d 100644 (file)
--- a/src/dygraph.js
+++ b/src/dygraph.js
@@ -1815,7 +1815,7 @@ Dygraph.prototype.updateSelection_ = function(opt_animFraction) {
     ctx.save();
     for (i = 0; i < this.selPoints_.length; i++) {
       var pt = this.selPoints_[i];
-      if (!utils.isOK(pt.canvasy)) continue;
+      if (isNaN(pt.canvasy) || pt.canvasy === null) continue;
 
       var circleSize = this.getNumericOption('highlightCircleSize', pt.name);
       var callback = this.getFunctionOption("drawHighlightPointCallback", pt.name);

diff --git a/src/plugins/legend.js b/src/plugins/legend.js
index c0ab35a..ab22fbb 100644 (file)
--- a/src/plugins/legend.js
+++ b/src/plugins/legend.js
@@ -268,7 +268,7 @@ Legend.generateLegendHTML = function(g, x, sel_points, oneEmWidth, row) {
       var seriesData = labelToSeries[pt.name];
       seriesData.y = pt.yval;
 
-      if ((pt.yval === 0 && !showZeros) || !utils.isOK(pt.canvasy)) {
+      if ((pt.yval === 0 && !showZeros) || isNaN(pt.canvasy) || pt.canvasy === null) {
         seriesData.isVisible = false;
         continue;
       }