@gravity-ui/chartkit 1.6.4 → 1.6.6

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## [1.6.6](https://github.com/gravity-ui/chartkit/compare/v1.6.5...v1.6.6) (2023-02-21)
+
+
+### Bug Fixes
+
+* add check if chart exists before adjust font size ([#128](https://github.com/gravity-ui/chartkit/issues/128)) ([451e3c1](https://github.com/gravity-ui/chartkit/commit/451e3c119858adffc02fede05eb9f91407b6e31c))
+
+## [1.6.5](https://github.com/gravity-ui/chartkit/compare/v1.6.4...v1.6.5) (2023-02-10)
+
+
+### Bug Fixes
+
+* **Highcharts plugin:** fix optional id usage for series identifier ([#125](https://github.com/gravity-ui/chartkit/issues/125)) ([f08b414](https://github.com/gravity-ui/chartkit/commit/f08b4145360db78c9efc6bd056772cd9f21a5fdb))
+
 ## [1.6.4](https://github.com/gravity-ui/chartkit/compare/v1.6.3...v1.6.4) (2023-02-09)
 
 

package/build/plugins/highcharts/__stories__/mocks/combo-chart-with-same-legend-titles.js

@@ -2,6 +2,7 @@ export const data = {
     data: {
         graphs: [
             {
+                id: 'uniq-1',
                 title: 'Office Supplies',
                 tooltip: {
                     chartKitFormatting: true,
@@ -90,6 +91,7 @@ export const data = {
                 type: 'line',
             },
             {
+                id: 'uniq-2',
                 title: 'Technology',
                 tooltip: {
                     chartKitFormatting: true,
@@ -136,6 +138,7 @@ export const data = {
                 type: 'line',
             },
             {
+                id: 'uniq-3',
                 title: 'Furniture',
                 tooltip: {
                     chartKitFormatting: true,
@@ -196,6 +199,7 @@ export const data = {
                 type: 'line',
             },
             {
+                id: 'uniq-4',
                 title: 'Furniture',
                 tooltip: {
                     chartKitFormatting: true,
@@ -256,6 +260,7 @@ export const data = {
                 stack: 'c44bfc70-a7a5-11ed-96a0-9bb7dddec993__',
             },
             {
+                id: 'uniq-5',
                 title: 'Office Supplies',
                 tooltip: {
                     chartKitFormatting: true,
@@ -344,6 +349,7 @@ export const data = {
                 stack: 'c44bfc70-a7a5-11ed-96a0-9bb7dddec993__',
             },
             {
+                id: 'uniq-6',
                 title: 'Technology',
                 tooltip: {
                     chartKitFormatting: true,

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

@@ -1078,7 +1078,7 @@ function fixTooltipOnClick(event, { options }) {
     fixTooltip(this.chart.tooltip, options, event);
 }
 function adjustDonutFontSize(chart, chartSeries, innerWidth, totals) {
-    if (!totals) {
+    if (!totals || !chart || !chartSeries) {
         return;
     }
     const MIN_ACCEPTABLE_INNER_SIZE = 400;

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

@@ -1,32 +1,32 @@
 import { drawComments, hideComments } from '../comments/drawing';
 import { isNavigatorSeries } from './utils';
-const getUniqueSeriesName = (series) => {
-    return `${series.name}__${series.index}`;
+const getSeriesIdentifier = (series) => {
+    return series.userOptions.id || series.name;
 };
 const needSetVisible = (seriesName, seriesVisible, chartSeries) => {
     if (!seriesVisible) {
         return false;
     }
     const hasAnotherVisibleSeries = chartSeries
-        .filter((series) => series.options.showInLegend !== false && getUniqueSeriesName(series) !== seriesName)
+        .filter((series) => series.options.showInLegend !== false && getSeriesIdentifier(series) !== seriesName)
         .some((series) => series.visible);
     return seriesVisible && !hasAnotherVisibleSeries;
 };
 const updateSeries = (series, chartSeries, type) => {
-    const clickedSeriesName = getUniqueSeriesName(series);
+    const clickedSeriesName = getSeriesIdentifier(series);
     switch (type) {
         case 'extended': {
             chartSeries.forEach((item) => {
-                if (getUniqueSeriesName(item) === clickedSeriesName) {
+                if (getSeriesIdentifier(item) === clickedSeriesName) {
                     item.setVisible(!item.visible, false);
                 }
             });
             break;
         }
         case 'default': {
-            const visible = needSetVisible(getUniqueSeriesName(series), series.visible, chartSeries);
+            const visible = needSetVisible(getSeriesIdentifier(series), series.visible, chartSeries);
             chartSeries.forEach((item) => {
-                if (getUniqueSeriesName(item) === clickedSeriesName) {
+                if (getSeriesIdentifier(item) === clickedSeriesName) {
                     item.setVisible(true, false);
                 }
                 else {

package/build/types/index.d.ts

@@ -27,10 +27,6 @@ export declare type ChartKitProps<T extends ChartKitType> = {
     data: ChartKitWidget[T]['data'];
     id?: string;
     isMobile?: boolean;
-    /**
-     * @depricated please use onRender & onChartLoad instead
-     * @param data
-     */
     onLoad?: (data?: ChartKitOnLoadData<T>) => void;
     /**
      * called on each render

package/package.json

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