From f0b4864495f404ffbd06dbbf428e57b20e918832 Mon Sep 17 00:00:00 2001 From: Akihiko Kusanagi Date: Thu, 25 Oct 2018 02:00:41 +0800 Subject: [PATCH] Add error margin for detecting if a point or line is in the chartArea --- src/core/core.scale.js | 6 ++++-- src/elements/element.point.js | 2 +- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/src/core/core.scale.js b/src/core/core.scale.js index 363815e63b6..3d734085851 100644 --- a/src/core/core.scale.js +++ b/src/core/core.scale.js @@ -710,6 +710,8 @@ module.exports = Element.extend({ var yTickStart = options.position === 'bottom' ? me.top + axisWidth : me.bottom - tl - axisWidth; var yTickEnd = options.position === 'bottom' ? me.top + axisWidth + tl : me.bottom + axisWidth; + var errMargin = 1.01; // 1.01 is margin for Accumulated error. (Especially Edge, IE.) + helpers.each(ticks, function(tick, index) { // autoskipper skipped this tick (#4635) if (helpers.isNullOrUndef(tick.label)) { @@ -753,7 +755,7 @@ module.exports = Element.extend({ } var xLineValue = getLineValue(me, index, gridLines.offsetGridLines && ticks.length > 1); - if (xLineValue < me.left) { + if (xLineValue < me.left / errMargin) { lineColor = 'rgba(0,0,0,0)'; } xLineValue += helpers.aliasPixel(lineWidth); @@ -780,7 +782,7 @@ module.exports = Element.extend({ labelX = isLeft ? me.right - labelXOffset : me.left + labelXOffset; var yLineValue = getLineValue(me, index, gridLines.offsetGridLines && ticks.length > 1); - if (yLineValue < me.top) { + if (yLineValue < me.top / errMargin) { lineColor = 'rgba(0,0,0,0)'; } yLineValue += helpers.aliasPixel(lineWidth); diff --git a/src/elements/element.point.js b/src/elements/element.point.js index 2bcdc88f0f8..67710b5e468 100644 --- a/src/elements/element.point.js +++ b/src/elements/element.point.js @@ -79,7 +79,7 @@ module.exports = Element.extend({ } // Clipping for Points. - if (chartArea === undefined || (model.x >= chartArea.left && chartArea.right * errMargin >= model.x && model.y >= chartArea.top && chartArea.bottom * errMargin >= model.y)) { + if (chartArea === undefined || (model.x >= chartArea.left / errMargin && chartArea.right * errMargin >= model.x && model.y >= chartArea.top / errMargin && chartArea.bottom * errMargin >= model.y)) { ctx.strokeStyle = vm.borderColor || defaultColor; ctx.lineWidth = helpers.valueOrDefault(vm.borderWidth, defaults.global.elements.point.borderWidth); ctx.fillStyle = vm.backgroundColor || defaultColor;