X-Git-Url: https://adrianiainlam.tk/git/?a=blobdiff_plain;f=dygraph-utils.js;h=c6c3a60345712b76cd023fa212011b3fb0611c17;hb=966ac3fa0de18a7b5ddefe4aba6938d8358ea0cf;hp=61aba43752f12a283b0802871a5f6e8495ca6eac;hpb=5879307d645ba06bceb1664d33f689888628556c;p=dygraphs.git diff --git a/dygraph-utils.js b/dygraph-utils.js index 61aba43..c6c3a60 100644 --- a/dygraph-utils.js +++ b/dygraph-utils.js @@ -332,6 +332,21 @@ Dygraph.isOK = function(x) { }; /** + * @private + * @param { Object } p The point to consider, valid points are {x, y} objects + * @param { Boolean } allowNaNY Treat point with y=NaN as valid + * @return { Boolean } Whether the point has numeric x and y. + */ +Dygraph.isValidPoint = function(p, allowNaNY) { + if (!p) return false; // null or undefined object + if (p.yval === null) return false; // missing point + if (p.x === null || p.x === undefined) return false; + if (p.y === null || p.y === undefined) return false; + if (isNaN(p.x) || (!allowNaNY && isNaN(p.y))) return false; + return true; +}; + +/** * Number formatting function which mimicks the behavior of %g in printf, i.e. * either exponential or fixed format (without trailing 0s) is used depending on * the length of the generated string. The advantage of this format is that @@ -479,9 +494,17 @@ Dygraph.dateParser = function(dateStr) { var dateStrSlashed; var d; - // Let the system try the format first. - d = Dygraph.dateStrToMillis(dateStr); - if (d && !isNaN(d)) return d; + // Let the system try the format first, with one caveat: + // YYYY-MM-DD[ HH:MM:SS] is interpreted as UTC by a variety of browsers. + // dygraphs displays dates in local time, so this will result in surprising + // inconsistencies. But if you specify "T" or "Z" (i.e. YYYY-MM-DDTHH:MM:SS), + // then you probably know what you're doing, so we'll let you go ahead. + // Issue: http://code.google.com/p/dygraphs/issues/detail?id=255 + if (dateStr.search("-") == -1 || + dateStr.search("T") != -1 || dateStr.search("Z") != -1) { + d = Dygraph.dateStrToMillis(dateStr); + if (d && !isNaN(d)) return d; + } if (dateStr.search("-") != -1) { // e.g. '2009-7-12' or '2009-07-12' dateStrSlashed = dateStr.replace("-", "/", "g"); @@ -844,14 +867,14 @@ Dygraph.regularShape_ = function( var coords = computeCoordinates(); ctx.lineTo(coords[0], coords[1]); } + ctx.fill(); ctx.stroke(); - ctx.closePath(); } Dygraph.shapeFunction_ = function(sides, rotationRadians, delta) { return function(g, name, ctx, cx, cy, color, radius) { - ctx.lineWidth = 1; ctx.strokeStyle = color; + ctx.fillStyle = "white"; Dygraph.regularShape_(ctx, sides, radius, cx, cy, rotationRadians, delta); }; }; @@ -875,12 +898,13 @@ Dygraph.Circles = { CIRCLE : function(g, name, ctx, cx, cy, color, radius) { ctx.beginPath(); ctx.strokeStyle = color; + ctx.fillStyle = "white"; ctx.arc(cx, cy, radius, 0, 2 * Math.PI, false); + ctx.fill(); ctx.stroke(); }, STAR : Dygraph.shapeFunction_(5, 0, 4 * Math.PI / 5), PLUS : function(g, name, ctx, cx, cy, color, radius) { - ctx.lineWidth = 1; ctx.strokeStyle = color; ctx.beginPath(); @@ -893,12 +917,10 @@ Dygraph.Circles = { ctx.moveTo(cx, cy + radius); ctx.lineTo(cx, cy - radius); ctx.closePath(); - ctx.stroke(); }, EX : function(g, name, ctx, cx, cy, color, radius) { - ctx.lineWidth = 1; - ctx.strokeStyle = "black"; + ctx.strokeStyle = color; ctx.beginPath(); ctx.moveTo(cx + radius, cy + radius); @@ -910,7 +932,6 @@ Dygraph.Circles = { ctx.moveTo(cx + radius, cy - radius); ctx.lineTo(cx - radius, cy + radius); ctx.closePath(); - ctx.stroke(); } };