@gravity-ui/chartkit 2.2.2 → 2.3.1

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 # Changelog
 
+## [2.3.1](https://github.com/gravity-ui/chartkit/compare/v2.3.0...v2.3.1) (2023-04-12)
+
+
+### Bug Fixes
+
+* **Highcharts plugin:** navigator series visibility on legend item click ([#148](https://github.com/gravity-ui/chartkit/issues/148)) ([ae7d38b](https://github.com/gravity-ui/chartkit/commit/ae7d38b3ffa8d4b037a762bf26f4e68b504c7886))
+
+## [2.3.0](https://github.com/gravity-ui/chartkit/compare/v2.2.2...v2.3.0) (2023-04-11)
+
+
+### Features
+
+* **Indicator plugin:** add onRender ([#145](https://github.com/gravity-ui/chartkit/issues/145)) ([3a044c0](https://github.com/gravity-ui/chartkit/commit/3a044c08583cd96bd08e3fb29a89c06ca9a6be18))
+* **Indicator plugin:** add renderTime ([3a044c0](https://github.com/gravity-ui/chartkit/commit/3a044c08583cd96bd08e3fb29a89c06ca9a6be18))
+
 ## [2.2.2](https://github.com/gravity-ui/chartkit/compare/v2.2.1...v2.2.2) (2023-03-28)
 
 

package/build/plugins/highcharts/renderer/helpers/config/handleLegendItemClick.js

@@ -5,6 +5,9 @@ const getSeriesIdentifier = (item) => {
     if (item instanceof Highcharts.Point) {
         return item.name;
     }
+    if (isNavigatorSeries(item) && item.baseSeries) {
+        return item.baseSeries.userOptions.id || item.baseSeries.name;
+    }
     return item.userOptions.id || item.name;
 };
 const needSetVisible = (seriesName, seriesVisible, chartSeries) => {

package/build/plugins/highcharts/types/highcharts-extends.d.ts

@@ -35,4 +35,10 @@ declare module 'highcharts' {
     }
 
     interface SeriesOptionsRegistry extends Record<string, Record<string, unknown>> {}
+
+    // for Stock chart from https://github.com/highcharts/highcharts/blob/master/ts/Stock/Navigator/NavigatorComposition.ts#L65
+    interface Series {
+        baseSeries?: Series;
+        navigatorSeries?: Series;
+    }
 }

package/build/plugins/indicator/renderer/IndicatorWidget.js

@@ -4,6 +4,7 @@ import { isEmpty } from 'lodash';
 import { i18n } from '../../../i18n';
 import { CHARTKIT_ERROR_CODE, ChartKitError } from '../../../libs';
 import { CHARTKIT_SCROLLABLE_NODE_CLASSNAME } from '../../../constants';
+import { getChartPerformanceDuration, getRandomCKId, markChartPerformance } from '../../../utils';
 import { IndicatorItem } from './IndicatorItem';
 import './IndicatorWidget.css';
 const b = block('chartkit-indicator');
@@ -11,12 +12,16 @@ const IndicatorWidget = React.forwardRef(
 // _ref needs to avoid this React warning:
 // "forwardRef render functions accept exactly two parameters: props and ref"
 function IndicatorWidgetInner(props, _ref) {
-    const { onLoad, formatNumber, data: { data = [], defaultColor }, } = props;
+    const { onLoad, formatNumber, data: { data = [], defaultColor }, id, onRender, } = props;
+    const generatedId = React.useMemo(() => `${id}_${getRandomCKId()}`, [data, defaultColor, formatNumber, id]);
+    markChartPerformance(generatedId);
     React.useLayoutEffect(() => {
-        // TODO: swap to onRender after https://github.com/gravity-ui/chartkit/issues/33
-        onLoad === null || onLoad === void 0 ? void 0 : onLoad();
-        // TODO: add onRender with renderTime Issue #114
-    });
+        if (onRender) {
+            onRender({ renderTime: getChartPerformanceDuration(generatedId) });
+            return;
+        }
+        onLoad === null || onLoad === void 0 ? void 0 : onLoad({ widgetRendering: getChartPerformanceDuration(generatedId) });
+    }, [onLoad, onRender, generatedId]);
     if (isEmpty(data)) {
         throw new ChartKitError({
             code: CHARTKIT_ERROR_CODE.NO_DATA,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gravity-ui/chartkit",
-  "version": "2.2.2",
+  "version": "2.3.1",
   "description": "React component used to render charts based on any sources you need",
   "license": "MIT",
   "repository": "git@github.com:gravity-ui/ChartKit.git",