fix(axis): always render tickLine unless visible is false (#2194)

BREAKING CHANGE: Now respects `tickLine.padding` whenever `tickLine.visible` is `true`
elastic · Oct 10, 2023 · ec95d50 · ec95d50
1 parent 132327d
commit ec95d50
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 7 deletions.
diff --git a/packages/charts/src/chart_types/xy_chart/annotations/rect/dimensions.ts b/packages/charts/src/chart_types/xy_chart/annotations/rect/dimensions.ts
@@ -243,10 +243,8 @@ function maxOf(base: number, value: number | string | null | undefined): number
   return typeof value === 'number' ? Math.max(value, base) : typeof value === 'string' ? value : base;
 }
 
-function getOutsideDimension(style: AxisStyle): number {
-  const { visible, size, strokeWidth } = style.tickLine;
-
-  return visible && size > 0 && strokeWidth > 0 ? size : 0;
+function getOutsideDimension({ tickLine: { visible, size } }: AxisStyle): number {
+  return visible ? size : 0;
 }
 
 /**

diff --git a/packages/charts/src/chart_types/xy_chart/utils/axis_utils.ts b/packages/charts/src/chart_types/xy_chart/utils/axis_utils.ts
@@ -24,7 +24,6 @@ import { Range } from '../../../utils/domain';
 import { AxisId } from '../../../utils/ids';
 import { Point } from '../../../utils/point';
 import { AxisStyle, TextAlignment, TextOffset, Theme } from '../../../utils/themes/theme';
-import { MIN_STROKE_WIDTH } from '../renderer/canvas/primitives/line';
 import { Projection } from '../state/selectors/visible_ticks';
 import { SeriesDomainsAndData } from '../state/utils/types';
 
@@ -291,8 +290,8 @@ export function getPosition(
 }
 
 /** @internal */
-export function shouldShowTicks({ visible, strokeWidth, size }: AxisStyle['tickLine'], axisHidden: boolean): boolean {
-  return !axisHidden && visible && size > 0 && strokeWidth >= MIN_STROKE_WIDTH;
+export function shouldShowTicks({ visible }: AxisStyle['tickLine'], axisHidden: boolean): boolean {
+  return !axisHidden && visible;
 }
 
 /** @internal */