@coinbase/cds-web-visualization 3.4.0-beta.2 → 3.4.0-beta.20

package/esm/chart/CartesianChart.js

@@ -1,4 +1,4 @@
-const _excluded = ["series", "children", "animate", "xAxis", "yAxis", "inset", "enableScrubbing", "onScrubberPositionChange", "width", "height", "className", "style"];
+const _excluded = ["series", "children", "animate", "xAxis", "yAxis", "inset", "enableScrubbing", "onScrubberPositionChange", "legend", "legendPosition", "legendAccessibilityLabel", "width", "height", "className", "classNames", "style", "styles", "accessibilityLabel"];
 function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
 function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { _defineProperty(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
 function _defineProperty(e, r, t) { return (r = _toPropertyKey(r)) in e ? Object.defineProperty(e, r, { value: t, enumerable: !0, configurable: !0, writable: !0 }) : e[r] = t, e; }
@@ -12,23 +12,30 @@ import { useDimensions } from '@coinbase/cds-web/hooks/useDimensions';
 import { Box } from '@coinbase/cds-web/layout';
 import { ScrubberProvider } from './scrubber/ScrubberProvider';
 import { CartesianChartProvider } from './ChartProvider';
+import { Legend } from './legend';
 import { defaultAxisId, defaultChartInset, getAxisConfig, getAxisDomain, getAxisRange, getAxisScale, getChartInset, getStackedSeriesData as calculateStackedSeriesData, useTotalAxisPadding } from './utils';
-import { jsx as _jsx } from "react/jsx-runtime";
+import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 const focusStylesCss = "cds-focusStylesCss-f4oy7ru";
 export const CartesianChart = /*#__PURE__*/memo(/*#__PURE__*/forwardRef((_ref, ref) => {
   let {
       series,
       children,
       animate = true,
-      xAxis: xAxisConfigInput,
-      yAxis: yAxisConfigInput,
-      inset: insetInput,
+      xAxis: xAxisConfigProp,
+      yAxis: yAxisConfigProp,
+      inset,
       enableScrubbing,
       onScrubberPositionChange,
+      legend,
+      legendPosition = 'bottom',
+      legendAccessibilityLabel,
       width = '100%',
       height = '100%',
       className,
-      style
+      classNames,
+      style,
+      styles,
+      accessibilityLabel
     } = _ref,
     props = _objectWithoutProperties(_ref, _excluded);
   const {
@@ -36,15 +43,13 @@ export const CartesianChart = /*#__PURE__*/memo(/*#__PURE__*/forwardRef((_ref, r
     width: chartWidth,
     height: chartHeight
   } = useDimensions();
-  const internalSvgRef = useRef(null);
-  const userInset = useMemo(() => {
-    return getChartInset(insetInput, defaultChartInset);
-  }, [insetInput]);
+  const svgRef = useRef(null);
+  const calculatedInset = useMemo(() => getChartInset(inset, defaultChartInset), [inset]);
 
   // Axis configs store the properties of each axis, such as id, scale type, domain limit, etc.
   // We only support 1 x axis but allow for multiple y axes.
-  const xAxisConfig = useMemo(() => getAxisConfig('x', xAxisConfigInput)[0], [xAxisConfigInput]);
-  const yAxisConfig = useMemo(() => getAxisConfig('y', yAxisConfigInput), [yAxisConfigInput]);
+  const xAxisConfig = useMemo(() => getAxisConfig('x', xAxisConfigProp)[0], [xAxisConfigProp]);
+  const yAxisConfig = useMemo(() => getAxisConfig('y', yAxisConfigProp), [yAxisConfigProp]);
   const {
     renderedAxes,
     registerAxis,
@@ -59,10 +64,10 @@ export const CartesianChart = /*#__PURE__*/memo(/*#__PURE__*/forwardRef((_ref, r
       height: 0
     };
     const totalInset = {
-      top: userInset.top + axisPadding.top,
-      right: userInset.right + axisPadding.right,
-      bottom: userInset.bottom + axisPadding.bottom,
-      left: userInset.left + axisPadding.left
+      top: calculatedInset.top + axisPadding.top,
+      right: calculatedInset.right + axisPadding.right,
+      bottom: calculatedInset.bottom + axisPadding.bottom,
+      left: calculatedInset.left + axisPadding.left
     };
     const availableWidth = chartWidth - totalInset.left - totalInset.right;
     const availableHeight = chartHeight - totalInset.top - totalInset.bottom;
@@ -72,11 +77,15 @@ export const CartesianChart = /*#__PURE__*/memo(/*#__PURE__*/forwardRef((_ref, r
       width: availableWidth > 0 ? availableWidth : 0,
       height: availableHeight > 0 ? availableHeight : 0
     };
-  }, [chartHeight, chartWidth, userInset, axisPadding]);
-
-  // Axes contain the config along with domain and range, which get calculated here.
-  const xAxis = useMemo(() => {
-    if (!chartRect || chartRect.width <= 0 || chartRect.height <= 0) return undefined;
+  }, [chartHeight, chartWidth, calculatedInset, axisPadding]);
+  const {
+    xAxis,
+    xScale
+  } = useMemo(() => {
+    if (!chartRect || chartRect.width <= 0 || chartRect.height <= 0) return {
+      xAxis: undefined,
+      xScale: undefined
+    };
     const domain = getAxisDomain(xAxisConfig, series !== null && series !== void 0 ? series : [], 'x');
     const range = getAxisRange(xAxisConfig, chartRect, 'x');
     const axisConfig = {
@@ -87,11 +96,43 @@ export const CartesianChart = /*#__PURE__*/memo(/*#__PURE__*/forwardRef((_ref, r
       categoryPadding: xAxisConfig.categoryPadding,
       domainLimit: xAxisConfig.domainLimit
     };
-    return axisConfig;
+
+    // Create the scale
+    const scale = getAxisScale({
+      config: axisConfig,
+      type: 'x',
+      range: axisConfig.range,
+      dataDomain: axisConfig.domain
+    });
+    if (!scale) return {
+      xAxis: undefined,
+      xScale: undefined
+    };
+
+    // Update axis config with actual scale domain (after .nice() or other adjustments)
+    const scaleDomain = scale.domain();
+    const actualDomain = Array.isArray(scaleDomain) && scaleDomain.length === 2 ? {
+      min: scaleDomain[0],
+      max: scaleDomain[1]
+    } : axisConfig.domain;
+    const finalAxisConfig = _objectSpread(_objectSpread({}, axisConfig), {}, {
+      domain: actualDomain
+    });
+    return {
+      xAxis: finalAxisConfig,
+      xScale: scale
+    };
   }, [xAxisConfig, series, chartRect]);
-  const yAxes = useMemo(() => {
+  const {
+    yAxes,
+    yScales
+  } = useMemo(() => {
     const axes = new Map();
-    if (!chartRect || chartRect.width <= 0 || chartRect.height <= 0) return axes;
+    const scales = new Map();
+    if (!chartRect || chartRect.width <= 0 || chartRect.height <= 0) return {
+      yAxes: axes,
+      yScales: scales
+    };
     yAxisConfig.forEach(axisParam => {
       var _axisParam$id, _series$filter, _axisParam$domainLimi;
       const axisId = (_axisParam$id = axisParam.id) !== null && _axisParam$id !== void 0 ? _axisParam$id : defaultAxisId;
@@ -102,36 +143,19 @@ export const CartesianChart = /*#__PURE__*/memo(/*#__PURE__*/forwardRef((_ref, r
         return ((_s$yAxisId = s.yAxisId) !== null && _s$yAxisId !== void 0 ? _s$yAxisId : defaultAxisId) === axisId;
       })) !== null && _series$filter !== void 0 ? _series$filter : [];
 
-      // Calculate domain and range in one pass
-      const domain = getAxisDomain(axisParam, relevantSeries, 'y');
+      // Calculate domain and range
+      const dataDomain = getAxisDomain(axisParam, relevantSeries, 'y');
       const range = getAxisRange(axisParam, chartRect, 'y');
-      axes.set(axisId, {
+      const axisConfig = {
         scaleType: axisParam.scaleType,
-        domain,
+        domain: dataDomain,
         range,
         data: axisParam.data,
         categoryPadding: axisParam.categoryPadding,
         domainLimit: (_axisParam$domainLimi = axisParam.domainLimit) !== null && _axisParam$domainLimi !== void 0 ? _axisParam$domainLimi : 'nice'
-      });
-    });
-    return axes;
-  }, [yAxisConfig, series, chartRect]);
+      };
 
-  // Scales are the functions that convert data values to visual positions.
-  // They are calculated here based on the above axes.
-  const xScale = useMemo(() => {
-    if (!chartRect || chartRect.width <= 0 || chartRect.height <= 0 || xAxis === undefined) return undefined;
-    return getAxisScale({
-      config: xAxis,
-      type: 'x',
-      range: xAxis.range,
-      dataDomain: xAxis.domain
-    });
-  }, [chartRect, xAxis]);
-  const yScales = useMemo(() => {
-    const scales = new Map();
-    if (!chartRect || chartRect.width <= 0 || chartRect.height <= 0) return scales;
-    yAxes.forEach((axisConfig, axisId) => {
+      // Create the scale
       const scale = getAxisScale({
         config: axisConfig,
         type: 'y',
@@ -140,10 +164,23 @@ export const CartesianChart = /*#__PURE__*/memo(/*#__PURE__*/forwardRef((_ref, r
       });
       if (scale) {
         scales.set(axisId, scale);
+
+        // Update axis config with actual scale domain (after .nice() or other adjustments)
+        const scaleDomain = scale.domain();
+        const actualDomain = Array.isArray(scaleDomain) && scaleDomain.length === 2 ? {
+          min: scaleDomain[0],
+          max: scaleDomain[1]
+        } : axisConfig.domain;
+        axes.set(axisId, _objectSpread(_objectSpread({}, axisConfig), {}, {
+          domain: actualDomain
+        }));
       }
     });
-    return scales;
-  }, [chartRect, yAxes]);
+    return {
+      yAxes: axes,
+      yScales: scales
+    };
+  }, [yAxisConfig, series, chartRect]);
   const getXAxis = useCallback(() => xAxis, [xAxis]);
   const getYAxis = useCallback(id => yAxes.get(id !== null && id !== void 0 ? id : defaultAxisId), [yAxes]);
   const getXScale = useCallback(() => xScale, [xScale]);
@@ -157,6 +194,20 @@ export const CartesianChart = /*#__PURE__*/memo(/*#__PURE__*/forwardRef((_ref, r
     if (!seriesId) return undefined;
     return stackedDataMap.get(seriesId);
   }, [stackedDataMap]);
+  const dataLength = useMemo(() => {
+    // If xAxis has categorical data, use that length
+    if (xAxisConfig.data && xAxisConfig.data.length > 0) {
+      return xAxisConfig.data.length;
+    }
+
+    // Otherwise, find the longest series
+    if (!series || series.length === 0) return 0;
+    return series.reduce((max, s) => {
+      var _seriesData$length;
+      const seriesData = getStackedSeriesData(s.id);
+      return Math.max(max, (_seriesData$length = seriesData === null || seriesData === void 0 ? void 0 : seriesData.length) !== null && _seriesData$length !== void 0 ? _seriesData$length : 0);
+    }, 0);
+  }, [xAxisConfig.data, series, getStackedSeriesData]);
   const getAxisBounds = useCallback(axisId => {
     const axis = renderedAxes.get(axisId);
     if (!axis || !chartRect) return;
@@ -168,7 +219,7 @@ export const CartesianChart = /*#__PURE__*/memo(/*#__PURE__*/forwardRef((_ref, r
     const offsetFromPreviousAxes = axesAtPosition.slice(0, axisIndex).reduce((sum, a) => sum + a.size, 0);
     if (axis.position === 'top') {
       // Position above the chart rect, accounting for user inset
-      const startY = userInset.top + offsetFromPreviousAxes;
+      const startY = calculatedInset.top + offsetFromPreviousAxes;
       return {
         x: chartRect.x,
         y: startY,
@@ -186,7 +237,7 @@ export const CartesianChart = /*#__PURE__*/memo(/*#__PURE__*/forwardRef((_ref, r
       };
     } else if (axis.position === 'left') {
       // Position to the left of the chart rect, accounting for user inset
-      const startX = userInset.left + offsetFromPreviousAxes;
+      const startX = calculatedInset.left + offsetFromPreviousAxes;
       return {
         x: startX,
         y: chartRect.y,
@@ -203,7 +254,7 @@ export const CartesianChart = /*#__PURE__*/memo(/*#__PURE__*/forwardRef((_ref, r
         height: chartRect.height
       };
     }
-  }, [renderedAxes, chartRect, userInset]);
+  }, [renderedAxes, chartRect, calculatedInset]);
   const contextValue = useMemo(() => ({
     series: series !== null && series !== void 0 ? series : [],
     getSeries,
@@ -216,43 +267,79 @@ export const CartesianChart = /*#__PURE__*/memo(/*#__PURE__*/forwardRef((_ref, r
     getXScale,
     getYScale,
     drawingArea: chartRect,
+    dataLength,
     registerAxis,
     unregisterAxis,
     getAxisBounds
-  }), [series, getSeries, getStackedSeriesData, animate, chartWidth, chartHeight, getXAxis, getYAxis, getXScale, getYScale, chartRect, registerAxis, unregisterAxis, getAxisBounds]);
-  return /*#__PURE__*/_jsx(Box, _objectSpread(_objectSpread({
+  }), [series, getSeries, getStackedSeriesData, animate, chartWidth, chartHeight, getXAxis, getYAxis, getXScale, getYScale, chartRect, dataLength, registerAxis, unregisterAxis, getAxisBounds]);
+  const rootClassNames = useMemo(() => cx(className, classNames === null || classNames === void 0 ? void 0 : classNames.root), [className, classNames]);
+  const rootStyles = useMemo(() => _objectSpread(_objectSpread({}, style), styles === null || styles === void 0 ? void 0 : styles.root), [style, styles === null || styles === void 0 ? void 0 : styles.root]);
+  const legendElement = useMemo(() => {
+    if (!legend) return;
+    if (legend === true) {
+      const isHorizontal = legendPosition === 'top' || legendPosition === 'bottom';
+      const flexDirection = isHorizontal ? 'row' : 'column';
+      return /*#__PURE__*/_jsx(Legend, {
+        accessibilityLabel: legendAccessibilityLabel,
+        flexDirection: flexDirection
+      });
+    }
+    return legend;
+  }, [legend, legendAccessibilityLabel, legendPosition]);
+  const rootBoxProps = useMemo(() => _objectSpread({
+    className: rootClassNames,
+    height,
+    style: rootStyles,
+    width
+  }, props), [rootClassNames, height, rootStyles, width, props]);
+  const chartContent = /*#__PURE__*/_jsx(Box, {
     ref: node => {
-      // Handle the observe ref, internal ref, and forwarded ref
       observe(node);
-      if (internalSvgRef.current !== node) {
-        internalSvgRef.current = node;
-      }
-      if (ref) {
-        if (typeof ref === 'function') {
-          ref(node);
-        } else {
-          ref.current = node;
-        }
-      }
     },
-    "aria-live": "polite",
-    as: "svg",
-    className: cx(enableScrubbing && focusStylesCss, className),
-    height: height,
-    role: "figure",
-    style: style,
-    tabIndex: enableScrubbing ? 0 : undefined,
-    width: width
-  }, props), {}, {
-    children: /*#__PURE__*/_jsx(CartesianChartProvider, {
-      value: contextValue,
-      children: /*#__PURE__*/_jsx(ScrubberProvider, {
-        enableScrubbing: !!enableScrubbing,
-        onScrubberPositionChange: onScrubberPositionChange,
-        svgRef: internalSvgRef,
-        children: children
-      })
+    height: legend ? undefined : height,
+    style: {
+      flex: 1,
+      minHeight: 0,
+      minWidth: 0
+    },
+    width: legend ? undefined : width,
+    children: /*#__PURE__*/_jsx(Box, {
+      ref: node => {
+        const svgElement = node;
+        svgRef.current = svgElement;
+        // Forward the ref to the user
+        if (ref) {
+          if (typeof ref === 'function') {
+            ref(svgElement);
+          } else {
+            ref.current = svgElement;
+          }
+        }
+      },
+      accessibilityLabel: accessibilityLabel,
+      "aria-live": "polite",
+      as: "svg",
+      className: cx(enableScrubbing && focusStylesCss, classNames === null || classNames === void 0 ? void 0 : classNames.chart),
+      height: "100%",
+      style: styles === null || styles === void 0 ? void 0 : styles.chart,
+      tabIndex: enableScrubbing ? 0 : undefined,
+      width: "100%",
+      children: children
+    })
+  });
+  return /*#__PURE__*/_jsx(CartesianChartProvider, {
+    value: contextValue,
+    children: /*#__PURE__*/_jsx(ScrubberProvider, {
+      enableScrubbing: !!enableScrubbing,
+      onScrubberPositionChange: onScrubberPositionChange,
+      svgRef: svgRef,
+      children: legend ? /*#__PURE__*/_jsxs(Box, _objectSpread(_objectSpread({}, rootBoxProps), {}, {
+        flexDirection: legendPosition === 'top' || legendPosition === 'bottom' ? 'column' : 'row',
+        children: [(legendPosition === 'top' || legendPosition === 'left') && legendElement, chartContent, (legendPosition === 'bottom' || legendPosition === 'right') && legendElement]
+      })) : /*#__PURE__*/_jsx(Box, _objectSpread(_objectSpread({}, rootBoxProps), {}, {
+        children: chartContent
+      }))
     })
-  }));
+  });
 }));
 import "./CartesianChart.css";

package/esm/chart/ChartProvider.js

@@ -1,9 +1,9 @@
 import { createContext, useContext } from 'react';
-const CartesianChartContext = /*#__PURE__*/createContext(undefined);
+export const CartesianChartContext = /*#__PURE__*/createContext(undefined);
 export const useCartesianChartContext = () => {
   const context = useContext(CartesianChartContext);
   if (!context) {
-    throw new Error('useCartesianChartContext must be used within a CartesianChart component. See http://cds.coinbase.com/components/graphs/CartesianChart.');
+    throw new Error('useCartesianChartContext must be used within a CartesianChart component. See https://cds.coinbase.com/components/charts/CartesianChart.');
   }
   return context;
 };

package/esm/chart/Path.js

@@ -1,4 +1,5 @@
-const _excluded = ["animate", "clipRect", "clipOffset", "d"];
+const _excluded = ["d", "initialPath", "transitions"],
+  _excluded2 = ["animate", "clipRect", "clipOffset", "d", "transitions", "transition"];
 function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
 function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { _defineProperty(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
 function _defineProperty(e, r, t) { return (r = _toPropertyKey(r)) in e ? Object.defineProperty(e, r, { value: t, enumerable: !0, configurable: !0, writable: !0 }) : e[r] = t, e; }
@@ -6,84 +7,88 @@ function _toPropertyKey(t) { var i = _toPrimitive(t, "string"); return "symbol"
 function _toPrimitive(t, r) { if ("object" != typeof t || !t) return t; var e = t[Symbol.toPrimitive]; if (void 0 !== e) { var i = e.call(t, r || "default"); if ("object" != typeof i) return i; throw new TypeError("@@toPrimitive must return a primitive value."); } return ("string" === r ? String : Number)(t); }
 function _objectWithoutProperties(e, t) { if (null == e) return {}; var o, r, i = _objectWithoutPropertiesLoose(e, t); if (Object.getOwnPropertySymbols) { var n = Object.getOwnPropertySymbols(e); for (r = 0; r < n.length; r++) o = n[r], -1 === t.indexOf(o) && {}.propertyIsEnumerable.call(e, o) && (i[o] = e[o]); } return i; }
 function _objectWithoutPropertiesLoose(r, e) { if (null == r) return {}; var t = {}; for (var n in r) if ({}.hasOwnProperty.call(r, n)) { if (-1 !== e.indexOf(n)) continue; t[n] = r[n]; } return t; }
-import 'd3-transition';
-import { memo, useCallback, useEffect, useId, useRef } from 'react';
-import { useValueChanges } from '@coinbase/cds-common/hooks/useValueChanges';
-import { interpolatePath } from 'd3-interpolate-path';
-import { select } from 'd3-selection';
+import { memo, useId, useMemo } from 'react';
 import { m as motion } from 'framer-motion';
+import { defaultPathEnterTransition } from './utils/path';
+import { defaultTransition, getTransition, usePathTransition } from './utils/transition';
 import { useCartesianChartContext } from './ChartProvider';
+
+/**
+ * Duration in seconds for path enter transition.
+ * @deprecated Use `transitions.enter` on the Path component instead.
+ */
 import { jsx as _jsx, Fragment as _Fragment, jsxs as _jsxs } from "react/jsx-runtime";
-export const Path = /*#__PURE__*/memo(_ref => {
+export const pathEnterTransitionDuration = 0.5;
+const AnimatedPath = /*#__PURE__*/memo(_ref => {
+  let {
+      d = '',
+      initialPath,
+      transitions
+    } = _ref,
+    pathProps = _objectWithoutProperties(_ref, _excluded);
+  const interpolatedPath = usePathTransition({
+    currentPath: d,
+    initialPath,
+    transitions
+  });
+  return /*#__PURE__*/_jsx(motion.path, _objectSpread({
+    d: interpolatedPath
+  }, pathProps));
+});
+export const Path = /*#__PURE__*/memo(_ref2 => {
   let {
       animate: animateProp,
       clipRect,
       clipOffset = 0,
-      d = ''
-    } = _ref,
-    pathProps = _objectWithoutProperties(_ref, _excluded);
-  const pathRef = useRef(null);
+      d = '',
+      transitions,
+      transition
+    } = _ref2,
+    pathProps = _objectWithoutProperties(_ref2, _excluded2);
   const clipPathId = useId();
   const context = useCartesianChartContext();
-  const rect = clipRect !== null && clipRect !== void 0 ? clipRect : context.drawingArea;
+  const rect = clipRect !== undefined ? clipRect : context.drawingArea;
   const animate = animateProp !== null && animateProp !== void 0 ? animateProp : context.animate;
-  const {
-    previousValue: previousPath,
-    newValue: newPath,
-    hasChanged,
-    addPreviousValue
-  } = useValueChanges(d);
-  const morphPath = useCallback(() => {
-    if (!pathRef.current || !newPath || !previousPath) return;
-    select(pathRef.current).transition().duration(300).attrTween('d', function tween() {
-      return interpolatePath(previousPath, newPath);
-    });
-  }, [previousPath, newPath]);
-  useEffect(() => {
-    addPreviousValue(newPath);
-    if (animate && hasChanged && previousPath) {
-      morphPath();
-    }
-  }, [addPreviousValue, newPath, animate, hasChanged, previousPath, morphPath]);
+  const enterTransition = useMemo(() => getTransition(transitions === null || transitions === void 0 ? void 0 : transitions.enter, animate, defaultPathEnterTransition), [animate, transitions === null || transitions === void 0 ? void 0 : transitions.enter]);
+  const updateTransition = useMemo(() => getTransition((transitions === null || transitions === void 0 ? void 0 : transitions.update) !== undefined ? transitions.update : transition, animate, defaultTransition), [animate, transitions === null || transitions === void 0 ? void 0 : transitions.update, transition]);
 
   // The clip offset provides extra padding to prevent path from being cut off
   // Area charts typically use offset=0 for exact clipping, while lines use offset=2 for breathing room
   const totalOffset = clipOffset * 2; // Applied on both sides
 
+  const clipPathAnimation = useMemo(() => {
+    if (rect === null) return;
+    return {
+      hidden: {
+        width: 0
+      },
+      visible: {
+        width: rect.width + totalOffset,
+        transition: enterTransition
+      }
+    };
+  }, [rect, totalOffset, enterTransition]);
+  const clipPath = useMemo(() => rect !== null ? "url(#".concat(clipPathId, ")") : undefined, [rect, clipPathId]);
   return /*#__PURE__*/_jsxs(_Fragment, {
-    children: [/*#__PURE__*/_jsx("defs", {
+    children: [rect !== null && /*#__PURE__*/_jsx("defs", {
       children: /*#__PURE__*/_jsx("clipPath", {
         id: clipPathId,
-        children: !animate ? /*#__PURE__*/_jsx("rect", {
-          height: rect.height + totalOffset,
-          width: rect.width + totalOffset,
-          x: rect.x - clipOffset,
-          y: rect.y - clipOffset
-        }) : /*#__PURE__*/_jsx(motion.rect, {
+        children: /*#__PURE__*/_jsx(motion.rect, {
           animate: "visible",
           height: rect.height + totalOffset,
           initial: "hidden",
-          variants: {
-            hidden: {
-              width: 0
-            },
-            visible: {
-              width: rect.width + totalOffset,
-              transition: {
-                type: 'spring',
-                duration: 1,
-                bounce: 0
-              }
-            }
-          },
+          variants: clipPathAnimation,
           x: rect.x - clipOffset,
           y: rect.y - clipOffset
         })
       })
-    }), /*#__PURE__*/_jsx("path", _objectSpread({
-      ref: pathRef,
-      clipPath: "url(#".concat(clipPathId, ")"),
-      d: d
+    }), /*#__PURE__*/_jsx(AnimatedPath, _objectSpread({
+      clipPath: clipPath,
+      d: d,
+      transitions: {
+        enter: enterTransition,
+        update: updateTransition
+      }
     }, pathProps))]
   });
 });