refactor(Charts - New): rename xIndex to dataIndex in onDataPointClick event (#479)

Author: vbersch

packages/charts/src/components/BarChart/BarChart.tsx

@@ -154,7 +154,7 @@ const BarChart: FC<BarChartProps> = forwardRef((props: BarChartProps, ref: Ref<a
               .find((key) => payload[key] === value),
             value,
             payload: payload.payload,
-            xIndex: i
+            dataIndex: i
           })
         );
       }

packages/charts/src/components/ColumnChart/ColumnChart.tsx

@@ -155,7 +155,7 @@ const ColumnChart: FC<ColumnChartProps> = forwardRef((props: ColumnChartProps, r
                 : payload[key] === payload.value && key !== 'value'
             )[0],
             value: payload.value.length ? payload.value[1] - payload.value[0] : payload.value,
-            xIndex: eventOrIndex,
+            dataIndex: eventOrIndex,
             payload: payload.payload
           })
         );

packages/charts/src/components/ComposedChart/index.tsx

@@ -173,7 +173,7 @@ const ComposedChart: FC<ComposedChartProps> = forwardRef((props: ComposedChartPr
         onDataPointClick(
           enrichEventWithDetails(event ?? eventOrIndex, {
             value: payload.value.length ? payload.value[1] - payload.value[0] : payload.value,
-            xIndex: payload.index ?? eventOrIndex,
+            dataIndex: payload.index ?? eventOrIndex,
             dataKey: payload.value.length
               ? Object.keys(payload).filter((key) =>
                   payload.value.length
@@ -192,7 +192,7 @@ const ComposedChart: FC<ComposedChartProps> = forwardRef((props: ComposedChartPr
             {
               value: eventOrIndex.value,
               dataKey: eventOrIndex.dataKey,
-              xIndex: eventOrIndex.index,
+              dataIndex: eventOrIndex.index,
               payload: eventOrIndex.payload
             }
           )

packages/charts/src/components/LineChart/LineChart.tsx

@@ -145,7 +145,7 @@ const LineChart: FC<LineChartProps> = forwardRef((props: LineChartProps, ref: Re
             {
               value: eventOrIndex.value,
               dataKey: eventOrIndex.dataKey,
-              xIndex: eventOrIndex.index,
+              dataIndex: eventOrIndex.index,
               payload: eventOrIndex.payload
             }
           )

packages/charts/src/components/MicroBarChart/MicroBarChart.tsx

@@ -107,7 +107,7 @@ const MicroBarChart: FC<MicroBarChartProps> = forwardRef((props: MicroBarChartPr
             )[0],
             value: e.value.length ? e.value[1] - e.value[0] : e.value,
             payload: e.payload,
-            xIndex: i
+            dataIndex: i
           })
         );
       }

packages/charts/src/components/PieChart/PieChart.tsx

@@ -103,7 +103,8 @@ const PieChart: FC<PieChartProps> = forwardRef((props: PieChartProps, ref: Ref<a
             value: payload.value,
             dataKey: measure.accessor,
             name: payload.name,
-            payload: payload.payload
+            payload: payload.payload,
+            dataIndex: index
           })
         );
       }

packages/charts/src/components/RadarChart/RadarChart.tsx

@@ -114,7 +114,7 @@ const RadarChart: FC<RadarChartProps> = forwardRef((props: RadarChartProps, ref:
               value: eventOrIndex.value,
               dataKey: eventOrIndex.dataKey,
               name: eventOrIndex.payload.label,
-              xIndex: eventOrIndex.index,
+              dataIndex: eventOrIndex.index,
               payload: eventOrIndex.payload
             }
           )

packages/charts/src/components/RadialChart/RadialChart.tsx

@@ -11,7 +11,7 @@ export interface RadialChartProps extends CommonProps {
   maxValue?: number;
   displayValue?: number | string;
   color?: CSSProperties['color'];
-  onDataPointClick?: (event: CustomEvent<{ value: unknown; payload: unknown; xIndex: number }>) => void;
+  onDataPointClick?: (event: CustomEvent<{ value: unknown; payload: unknown; dataIndex: number }>) => void;
 }
 
 /**
@@ -34,7 +34,7 @@ const RadialChart: FC<RadialChartProps> = forwardRef((props: RadialChartProps, r
           enrichEventWithDetails(event, {
             value: payload.value,
             payload: payload.payload,
-            xIndex: i
+            dataIndex: i
           })
         );
       }