X-Git-Url: https://adrianiainlam.tk/git/?a=blobdiff_plain;f=dygraph-canvas.js;h=0c44e8479ef4ba71ba5a3b535111725d2c9257d2;hb=cbfcf21db4a2ef0f5db409591146f975ffe7b7f2;hp=ff291e5db633874c829342faca327855505a1a48;hpb=349dd9bae6482cef4fd2025a9feb1af776f432dc;p=dygraphs.git diff --git a/dygraph-canvas.js b/dygraph-canvas.js index ff291e5..0c44e84 100644 --- a/dygraph-canvas.js +++ b/dygraph-canvas.js @@ -661,11 +661,12 @@ DygraphCanvasRenderer.prototype._renderAnnotations = function() { } }; -DygraphCanvasRenderer.makeNextPointStep_ = function(connect, points, end) { +DygraphCanvasRenderer.makeNextPointStep_ = function( + connect, points, start, end) { if (connect) { return function(j) { - while (++j < end) { - if (!(points[j].yval === null)) break; + while (++j + start < end) { + if (!(points[start + j].yval === null)) break; } return j; } @@ -688,17 +689,22 @@ DygraphCanvasRenderer.prototype._drawStyledLine = function( var points = this.layout.points; var prevX = null; var prevY = null; + var nextY = null; var pointsOnLine = []; // Array of [canvasx, canvasy] pairs. if (!Dygraph.isArrayLike(strokePattern)) { strokePattern = null; } + var drawGapPoints = this.dygraph_.attr_('drawGapEdgePoints', setName); - var point; + var point, nextPoint; var next = DygraphCanvasRenderer.makeNextPointStep_( - this.attr_('connectSeparatedPoints'), points, afterLastIndexInSet); + this.attr_('connectSeparatedPoints'), points, firstIndexInSet, + afterLastIndexInSet); ctx.save(); - for (var j = firstIndexInSet; j < afterLastIndexInSet; j = next(j)) { - point = points[j]; + for (var j = 0; j < setLength; j = next(j)) { + point = points[firstIndexInSet + j]; + nextY = (next(j) < setLength) ? + points[firstIndexInSet + next(j)].canvasy : null; if (isNullOrNaN(point.canvasy)) { if (stepPlot && prevX !== null) { // Draw a horizontal line to the start of the missing data @@ -713,8 +719,15 @@ DygraphCanvasRenderer.prototype._drawStyledLine = function( } else { // A point is "isolated" if it is non-null but both the previous // and next points are null. - var isIsolated = (!prevX && (j == points.length - 1 || - isNullOrNaN(points[j+1].canvasy))); + var isIsolated = (!prevX && isNullOrNaN(nextY)); + if (drawGapPoints) { + // Also consider a point to be is "isolated" if it's adjacent to a + // null point, excluding the graph edges. + if ((j > 0 && !prevX) || + (next(j) < setLength && isNullOrNaN(nextY))) { + isIsolated = true; + } + } if (prevX === null) { prevX = point.canvasx; prevY = point.canvasy;