[chart] Make smarter default domain limit

Author: alexfauquette

packages/x-charts/src/internals/plugins/featurePlugins/useChartCartesianAxis/computeAxisValue.ts

@@ -26,6 +26,7 @@ import { ComputedAxisConfig, DefaultizedZoomOptions } from './useChartCartesianA
 import { ProcessedSeries } from '../../corePlugins/useChartSeries/useChartSeries.types';
 import { GetZoomAxisFilters, ZoomData } from './zoom.types';
 import { getAxisTriggerTooltip } from './getAxisTriggerTooltip';
+import { getAxisDomainLimit } from './getAxisDomainLimit';
 
 function getRange(
   drawingArea: ChartDrawingArea,
@@ -180,7 +181,7 @@ export function computeAxisValue<T extends ChartSeriesType>({
 
     const scaleType = axis.scaleType ?? ('linear' as const);
 
-    const domainLimit = axis.domainLimit ?? 'nice';
+    const domainLimit = getAxisDomainLimit(axis, axisDirection, axisIndex, formattedSeries);
 
     const axisExtremums = [axis.min ?? minData, axis.max ?? maxData];
 

packages/x-charts/src/internals/plugins/featurePlugins/useChartCartesianAxis/getAxisDomainLimit.ts

@@ -0,0 +1,26 @@
+import { AxisConfig } from '../../../../models/axis';
+import { CartesianChartSeriesType } from '../../../../models/seriesType/config';
+import { ProcessedSeries } from '../../corePlugins/useChartSeries';
+
+export const getAxisDomainLimit = <T extends CartesianChartSeriesType>(
+  axis: AxisConfig,
+  axisDirection: 'x' | 'y',
+  axisIndex: number,
+  formattedSeries: ProcessedSeries<T | 'line' | 'bar'>,
+): 'nice' | 'strict' | ((min: number, max: number) => { min: number; max: number }) => {
+  if (axis.domainLimit !== undefined) {
+    return axis.domainLimit;
+  }
+
+  for (const seriesId of formattedSeries.line?.seriesOrder ?? []) {
+    const series = formattedSeries.line!.series[seriesId];
+
+    if (
+      series.xAxisId === axis.id ||
+      (series.xAxisId === undefined && axisIndex === 0 && axisDirection === 'x')
+    ) {
+      return 'strict';
+    }
+  }
+  return 'nice';
+};