@pushwoosh/kit-charts 0.4.1 → 0.4.3

package/core/Chart/Chart.js

@@ -1,5 +1,5 @@
 import { jsx as _jsx, Fragment as _Fragment, jsxs as _jsxs } from "react/jsx-runtime";
-import { useCallback, useEffect, useMemo, useRef, useState } from 'react';
+import { useCallback, useEffect, useMemo, useState } from 'react';
 import { Grid, XAxis, XAxisLine, YAxis } from './Axes';
 import { ChartErrorBoundary } from './ChartErrorBoundary';
 import { Legend } from './Legend';
@@ -158,11 +158,13 @@ function ChartBody({
   const identity = seriesIdentity(options);
   // A different set of series is a different chart: start from its own options
   // again rather than carrying the previous chart's switched-off indices over.
-  useEffect(() => {
+  const identityDeps = [identity];
+  const [prevIdentityDeps, setPrevIdentityDeps] = useState(null);
+  if (prevIdentityDeps === null || identityDeps.some((dep, index) => dep !== prevIdentityDeps[index])) {
+    setPrevIdentityDeps(identityDeps);
     setHidden(hiddenByOptions(options));
     setZoom(null);
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [identity]);
+  }
   const toggle = useCallback(item => {
     // A tooltip captured before the toggle would keep quoting what just went away.
     setHover(null);
@@ -209,12 +211,10 @@ function ChartBody({
     hidden
   }) : null, [zoomed, canvas, registry, hidden]);
   const anyShown = built?.series.some(series => !series.hidden) ?? false;
-  const lastShown = useRef(null);
-  useEffect(() => {
-    if (built && anyShown) {
-      lastShown.current = built;
-    }
-  }, [built, anyShown]);
+  const [lastShown, setLastShown] = useState(null);
+  if (built && anyShown && built !== lastShown) {
+    setLastShown(built);
+  }
   // Switching off the last series must not reshape the axes under it: the plot
   // holds the geometry of the last state that had something on show, so the series
   // fades out where it stood instead of jumping to a rescaled axis first.
@@ -222,7 +222,7 @@ function ChartBody({
     if (anyShown || !built) {
       return built;
     }
-    const frozen = lastShown.current;
+    const frozen = lastShown;
     if (!frozen) {
       return built;
     }
@@ -235,7 +235,7 @@ function ChartBody({
       legend: built.legend,
       titles: built.titles
     };
-  }, [built, anyShown]);
+  }, [built, anyShown, lastShown]);
   const contextFor = useCallback(series => {
     if (!model) {
       return null;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pushwoosh/kit-charts",
-  "version": "0.4.1",
+  "version": "0.4.3",
   "description": "Small React chart library on SVG, with a Highcharts-shaped configuration model",
   "license": "MIT",
   "repository": {