From 0e5ce3f4e7ce8d6ab414377562b506d3d00bb512 Mon Sep 17 00:00:00 2001 From: andig Date: Sat, 19 Aug 2017 15:03:11 +0200 Subject: [PATCH] Fix autoskipped ticks messing up category axis (#4649) --- src/core/core.scale.js | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/core/core.scale.js b/src/core/core.scale.js index dd99d0bdd70..6f63b92dbd7 100644 --- a/src/core/core.scale.js +++ b/src/core/core.scale.js @@ -626,7 +626,8 @@ module.exports = function(Chart) { // Since we always show the last tick,we need may need to hide the last shown one before shouldSkip = (skipRatio > 1 && i % skipRatio > 0) || (i % skipRatio === 0 && i + skipRatio >= tickCount); if (shouldSkip && i !== tickCount - 1 || helpers.isNullOrUndef(tick.label)) { - continue; + // leave tick in place but make sure it's not displayed (#4635) + delete tick.label; } result.push(tick); } @@ -672,6 +673,11 @@ module.exports = function(Chart) { var yTickEnd = options.position === 'bottom' ? me.top + tl : me.bottom; helpers.each(ticks, function(tick, index) { + // autoskipper skipped this tick (#4635) + if (tick.label === undefined) { + return; + } + var label = tick.label; var lineWidth, lineColor, borderDash, borderDashOffset; if (index === (typeof me.zeroLineIndex !== 'undefined' ? me.zeroLineIndex : 0) && (options.offset === gridLines.offsetGridLines)) {