WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Commit e820ac9

Browse files
authored
Merge pull request #5658 from ag-grid/imoses/ag-15010b
AG-15010: Limit ordinal time scale max ticks
2 parents 00c4a5e + 8769a7d commit e820ac9

5 files changed

+3
-1
lines changed

packages/ag-charts-community/src/chart/axis/generateTicks.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,9 @@ function estimateScaleTickCount<TScale extends Scale<TDatum, number, TickInterva
139139
const zoomExtent = findRangeExtent(visibleRange);
140140

141141
if (CategoryScale.is(scale) || OrdinalTimeScale.is(scale)) {
142-
const maxTickCount = domain.length;
142+
const maxTickCount = CategoryScale.is(scale)
143+
? domain.length
144+
: Math.min(domain.length, Math.max(1, Math.floor(rangeExtent / (zoomExtent * defaultTickMinSpacing))));
143145
const estimatedTickCount = Math.ceil(rangeExtent / (zoomExtent * label.fontSize));
144146
return {
145147
tickCount: Math.min(estimatedTickCount, maxTickCount),
-278 Bytes
Loading
-278 Bytes
Loading

0 commit comments

Comments
 (0)