Skip to content

Commit 86b3461

Browse files
committed
move to plugin
1 parent 12e239f commit 86b3461

File tree

2 files changed

+8
-18
lines changed

2 files changed

+8
-18
lines changed

packages/x-charts/src/ChartsAxis/ChartsAxis.tsx

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ import {
1111
ChartsYAxisProps,
1212
} from '../models/axis';
1313
import { useXAxes, useYAxes } from '../hooks';
14-
import { useChartContext } from '../context/ChartProvider';
1514

1615
export interface ChartsAxisProps {
1716
/**
@@ -90,7 +89,6 @@ function ChartsAxis(props: ChartsAxisProps) {
9089
const { topAxis, leftAxis, rightAxis, bottomAxis, slots, slotProps } = props;
9190
const { xAxis, xAxisIds } = useXAxes();
9291
const { yAxis, yAxisIds } = useYAxes();
93-
const { instance } = useChartContext();
9492

9593
const leftId = getAxisId(leftAxis === undefined ? yAxisIds[0] : leftAxis, yAxisIds[0]);
9694
const bottomId = getAxisId(bottomAxis === undefined ? xAxisIds[0] : bottomAxis, xAxisIds[0]);
@@ -134,21 +132,6 @@ function ChartsAxis(props: ChartsAxisProps) {
134132
const leftAxisProps = mergeProps(leftAxis, slots, slotProps);
135133
const rightAxisProps = mergeProps(rightAxis, slots, slotProps);
136134

137-
React.useEffect(() => {
138-
if (topId) {
139-
instance.addAxisSide('top');
140-
}
141-
if (bottomId) {
142-
instance.addAxisSide('bottom');
143-
}
144-
if (leftId) {
145-
instance.addAxisSide('left');
146-
}
147-
if (rightId) {
148-
instance.addAxisSide('right');
149-
}
150-
}, [topId, bottomId, leftId, rightId, instance]);
151-
152135
return (
153136
<React.Fragment>
154137
{topId && <ChartsXAxis {...topAxisProps} position="top" axisId={topId} />}

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

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ import { defaultizeAxis } from './defaultizeAxis';
1212

1313
export const useChartCartesianAxis: ChartPlugin<
1414
UseChartCartesianAxisSignature<ChartSeriesType>
15-
> = ({ params, store, seriesConfig }) => {
15+
> = ({ params, store, seriesConfig, instance }) => {
1616
const { xAxis, yAxis, dataset } = params;
1717

1818
if (process.env.NODE_ENV !== 'production') {
@@ -31,6 +31,13 @@ export const useChartCartesianAxis: ChartPlugin<
3131
}
3232
}
3333

34+
React.useEffect(() => {
35+
instance.addAxisSide('top');
36+
instance.addAxisSide('bottom');
37+
instance.addAxisSide('left');
38+
instance.addAxisSide('right');
39+
}, [instance]);
40+
3441
const drawingArea = useSelector(store, selectorChartDrawingArea);
3542
const formattedSeries = useSelector(store, selectorChartSeriesState);
3643

0 commit comments

Comments
 (0)