Update dependency @elastic/charts to v25 (#91557)

package.json

@@ -351,7 +351,7 @@
     "@cypress/webpack-preprocessor": "^5.5.0",
     "@elastic/apm-rum": "^5.6.1",
     "@elastic/apm-rum-react": "^1.2.5",
-    "@elastic/charts": "24.6.0",
+    "@elastic/charts": "25.0.1",
     "@elastic/eslint-config-kibana": "link:packages/elastic-eslint-config-kibana",
     "@elastic/eslint-plugin-eui": "0.0.2",
     "@elastic/github-checks-reporter": "0.0.20b3",

src/plugins/charts/public/static/utils/transform_click_event.ts

@@ -30,9 +30,6 @@ export interface BrushTriggerEvent {
 
 type AllSeriesAccessors = Array<[accessor: Accessor | AccessorFn, value: string | number]>;
 
-// TODO: replace when exported from elastic/charts
-const DEFAULT_SINGLE_PANEL_SM_VALUE = '__ECH_DEFAULT_SINGLE_PANEL_SM_VALUE__';
-
 /**
  * returns accessor value from string or function accessor
  * @param datum
@@ -97,11 +94,11 @@ function getSplitChartValue({
   | string
   | number
   | undefined {
-  if (smHorizontalAccessorValue !== DEFAULT_SINGLE_PANEL_SM_VALUE) {
+  if (smHorizontalAccessorValue !== undefined) {
     return smHorizontalAccessorValue;
   }
 
-  if (smVerticalAccessorValue !== DEFAULT_SINGLE_PANEL_SM_VALUE) {
+  if (smVerticalAccessorValue !== undefined) {
     return smVerticalAccessorValue;
   }
 

src/plugins/vis_type_xy/public/components/detailed_tooltip.tsx

@@ -27,9 +27,6 @@ interface TooltipData {
   value: string;
 }
 
-// TODO: replace when exported from elastic/charts
-const DEFAULT_SINGLE_PANEL_SM_VALUE = '__ECH_DEFAULT_SINGLE_PANEL_SM_VALUE__';
-
 export const getTooltipData = (
   aspects: Aspects,
   header: TooltipValue | null,
@@ -81,7 +78,7 @@ export const getTooltipData = (
   if (
     aspects.splitColumn &&
     valueSeries.smHorizontalAccessorValue !== undefined &&
-    valueSeries.smHorizontalAccessorValue !== DEFAULT_SINGLE_PANEL_SM_VALUE
+    valueSeries.smHorizontalAccessorValue !== undefined
   ) {
     data.push({
       label: aspects.splitColumn.title,
@@ -92,7 +89,7 @@ export const getTooltipData = (
   if (
     aspects.splitRow &&
     valueSeries.smVerticalAccessorValue !== undefined &&
-    valueSeries.smVerticalAccessorValue !== DEFAULT_SINGLE_PANEL_SM_VALUE
+    valueSeries.smVerticalAccessorValue !== undefined
   ) {
     data.push({
       label: aspects.splitRow.title,

src/plugins/vis_type_xy/public/utils/get_legend_actions.tsx

@@ -20,7 +20,7 @@ export const getLegendActions = (
   onFilter: (data: ClickTriggerEvent, negate?: any) => void,
   getSeriesName: (series: XYChartSeriesIdentifier) => SeriesName
 ): LegendAction => {
-  return ({ series: xySeries }) => {
+  return ({ series: [xySeries] }) => {
     const [popoverOpen, setPopoverOpen] = useState(false);
     const [isfilterable, setIsfilterable] = useState(false);
     const series = xySeries as XYChartSeriesIdentifier;

src/plugins/vis_type_xy/public/utils/use_color_picker.tsx

@@ -36,7 +36,7 @@ export const useColorPicker = (
   getSeriesName: (series: XYChartSeriesIdentifier) => SeriesName
 ): LegendColorPicker =>
   useMemo(
-    () => ({ anchor, color, onClose, onChange, seriesIdentifier }) => {
+    () => ({ anchor, color, onClose, onChange, seriesIdentifiers: [seriesIdentifier] }) => {
       const seriesName = getSeriesName(seriesIdentifier as XYChartSeriesIdentifier);
       const handlChange = (newColor: string | null, event: BaseSyntheticEvent) => {
         if (!seriesName) {

src/plugins/vis_type_xy/public/vis_component.tsx

@@ -157,17 +157,12 @@ const VisComponent = (props: VisComponentProps) => {
     (
       visData: Datatable,
       xAccessor: Accessor | AccessorFn,
-      splitSeriesAccessors: Array<Accessor | AccessorFn>,
-      splitChartAccessor?: Accessor | AccessorFn
+      splitSeriesAccessors: Array<Accessor | AccessorFn>
     ) => {
       const splitSeriesAccessorFnMap = getSplitSeriesAccessorFnMap(splitSeriesAccessors);
       return (series: XYChartSeriesIdentifier): ClickTriggerEvent | null => {
         if (xAccessor !== null) {
-          return getFilterFromSeriesFn(visData)(
-            series,
-            splitSeriesAccessorFnMap,
-            splitChartAccessor
-          );
+          return getFilterFromSeriesFn(visData)(series, splitSeriesAccessorFnMap);
         }
 
         return null;
@@ -373,12 +368,7 @@ const VisComponent = (props: VisComponentProps) => {
             config.aspects.series && (config.aspects.series?.length ?? 0) > 0
               ? getLegendActions(
                   canFilter,
-                  getFilterEventData(
-                    visData,
-                    xAccessor,
-                    splitSeriesAccessors,
-                    splitChartColumnAccessor ?? splitChartRowAccessor
-                  ),
+                  getFilterEventData(visData, xAccessor, splitSeriesAccessors),
                   handleFilterAction,
                   getSeriesName
                 )

x-pack/plugins/uptime/public/components/common/charts/duration_chart.tsx

@@ -15,8 +15,8 @@ import {
   Position,
   timeFormatter,
   Settings,
-  SeriesIdentifier,
   BrushEndListener,
+  LegendItemListener,
 } from '@elastic/charts';
 import { getChartDateLabel } from '../../../lib/helper';
 import { LocationDurationLine } from '../../../../common/types';
@@ -75,7 +75,7 @@ export const DurationChartComponent = ({
     });
   };
 
-  const legendToggleVisibility = (legendItem: SeriesIdentifier | null) => {
+  const legendToggleVisibility: LegendItemListener = ([legendItem]) => {
     if (legendItem) {
       setHiddenLegends((prevState) => {
         if (prevState.includes(legendItem.specId)) {

yarn.lock

@@ -2138,10 +2138,10 @@
   dependencies:
     object-hash "^1.3.0"
 
-"@elastic/charts@24.6.0":
-  version "24.6.0"
-  resolved "https://registry.yarnpkg.com/@elastic/charts/-/charts-24.6.0.tgz#2123c72e69e1e4557be41ae55c085a5a9f75d3b6"
-  integrity sha512-fL0301EcHxJEYRzdlD4JIA3VXY4qwRPSkRrk8hvJNryTlQWEdyXZF3HNczk0IrgST5cfCOGAWG8IVtO59HxUJw==
+"@elastic/charts@25.0.1":
+  version "25.0.1"
+  resolved "https://registry.yarnpkg.com/@elastic/charts/-/charts-25.0.1.tgz#7c61fc22887b7b1feba3e52fc1b44f21a9c1d0bc"
+  integrity sha512-UYGO9Yg2+cdJOOs9DjlYeB2Oy/3UMXTtF6/yWZt2iXh7mmW9jI5DqfjgG/9BFSCwQZHUKBZ58d5ZCQfe72maGA==
   dependencies:
     "@popperjs/core" "^2.4.0"
     chroma-js "^2.1.0"