Skip to content

Commit d557224

Browse files
authored
[charts] Refactor optional chaining for props in PieChart, PieChartPro, and ScatterChartPro components (#19292)
1 parent d4a395e commit d557224

File tree

4 files changed

+7
-7
lines changed

4 files changed

+7
-7
lines changed

packages/x-charts-pro/src/PieChartPro/PieChartPro.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,13 +98,13 @@ const PieChartPro = React.forwardRef<SVGSVGElement, PieChartProProps>(
9898
<ChartDataProviderPro<'pie', PieChartProPluginSignatures> {...chartDataProviderProProps}>
9999
<ChartsWrapper
100100
legendPosition={props.slotProps?.legend?.position}
101-
legendDirection={props?.slotProps?.legend?.direction ?? 'vertical'}
101+
legendDirection={props.slotProps?.legend?.direction ?? 'vertical'}
102102
sx={sx}
103103
>
104104
{showToolbar ? <Toolbar /> : null}
105105
{!hideLegend && (
106106
<ChartsLegend
107-
direction={props?.slotProps?.legend?.direction ?? 'vertical'}
107+
direction={props.slotProps?.legend?.direction ?? 'vertical'}
108108
slots={slots}
109109
slotProps={slotProps}
110110
/>

packages/x-charts-pro/src/ScatterChartPro/ScatterChartPro.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ const ScatterChartPro = React.forwardRef(function ScatterChartPro(
135135
<ChartsAxisHighlight {...axisHighlightProps} />
136136
{children}
137137
</ChartsSurface>
138-
{!props.loading && <Tooltip trigger="item" {...props?.slotProps?.tooltip} />}
138+
{!props.loading && <Tooltip trigger="item" {...props.slotProps?.tooltip} />}
139139
</ChartsWrapper>
140140
</ChartDataProviderPro>
141141
);

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -144,13 +144,13 @@ const PieChart = React.forwardRef(function PieChart(
144144
<ChartDataProvider<'pie', PieChartPluginSignatures> {...chartDataProviderProps}>
145145
<ChartsWrapper
146146
legendPosition={props.slotProps?.legend?.position}
147-
legendDirection={props?.slotProps?.legend?.direction ?? 'vertical'}
147+
legendDirection={props.slotProps?.legend?.direction ?? 'vertical'}
148148
sx={sx}
149149
>
150150
{showToolbar && Toolbar ? <Toolbar {...props.slotProps?.toolbar} /> : null}
151151
{!hideLegend && (
152152
<ChartsLegend
153-
direction={props?.slotProps?.legend?.direction ?? 'vertical'}
153+
direction={props.slotProps?.legend?.direction ?? 'vertical'}
154154
slots={slots}
155155
slotProps={slotProps}
156156
/>

packages/x-charts/src/hooks/animation/useAnimatePieArcLabel.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -71,8 +71,8 @@ export function useAnimatePieArcLabel(
7171
{
7272
startAngle: props.startAngle,
7373
endAngle: props.endAngle,
74-
innerRadius: props?.arcLabelRadius ?? props.innerRadius,
75-
outerRadius: props?.arcLabelRadius ?? props.outerRadius,
74+
innerRadius: props.arcLabelRadius ?? props.innerRadius,
75+
outerRadius: props.arcLabelRadius ?? props.outerRadius,
7676
paddingAngle: props.paddingAngle,
7777
cornerRadius: props.cornerRadius,
7878
},

0 commit comments

Comments
 (0)