AI feedback

Author: alexfauquette

docs/data/charts/dataset/dataset.md

@@ -68,7 +68,7 @@ These series types require `datasetKeys` or the `valueGetter` to return values i
 
 ### Scatter series
 
-The scatter series requires `x` and `y` values, and optionally `colorValue` and `id` values.
+The scatter series requires `x` and `y` values, and optionally `colorValue`, `sizeValue`, and `id` values.
 You can provide these values with `datasetKeys` or with `valueGetter`.
 
 ```tsx

docs/data/charts/scatter/scatter.md

@@ -33,7 +33,7 @@ If your data is stored in an array of objects, you can use the `dataset` helper
 It accepts an array of objects such as `dataset={[{a: 1, b: 32, c: 873}, {a: 2, b: 41, c: 182}, ...]}`.
 
 Scatter series use a different pattern than other charts: use the `datasetKeys` property with an object that has required `x` and `y` keys.
-You can also include optional `id` and `color` keys.
+You can also include optional `id`, `color`, and `size` keys.
 
 See the [Dataset](/x/react-charts/dataset/) page to learn more.
 

packages/x-charts/src/ChartsTooltip/useItemTooltip.tsx

@@ -71,7 +71,7 @@ export function useInternalItemTooltip<SeriesType extends ChartSeriesType>():
 
   const colorAxisId: AxisId | undefined =
     'colorAxisId' in itemSeries || 'zAxisId' in itemSeries
-      ? (itemSeries.colorAxisId ?? itemSeries.zAxisId)
+      ? (itemSeries.colorAxisId ?? itemSeries.zAxisId ?? zAxisIds[0])
       : zAxisIds[0];
 
   const mainAxis =

packages/x-charts/src/models/seriesType/scatter.ts

@@ -69,7 +69,7 @@ export interface ScatterSeriesType
    * The keys used to retrieve data from the dataset.
    *
    * When this prop is provided, both `x` and `y` must be provided.
-   * While `color` and `id` are optional.
+   * While `color`, `size`, and `id` are optional.
    */
   datasetKeys?: {
     /**

packages/x-charts/src/models/z-axis.ts

@@ -31,11 +31,11 @@ export interface ZAxisConfig<V = any> {
    */
   max?: number;
   /**
-   * The config that defined how the values should be mapped to colors.
+   * The config that defines how the values should be mapped to colors.
    */
   colorMap?: OrdinalColorConfig | ContinuousColorConfig | PiecewiseColorConfig;
   /**
-   * The config that defined how the values should be mapped to sizes.
+   * The config that defines how the values should be mapped to sizes.
    */
   sizeMap?: OrdinalSizeConfig | ContinuousSizeConfig | PiecewiseSizeConfig;
 }