Merge pull request #2810 from VisActor/2178-bug-pivotchart-pie-select…

…ed-selected_reverse 2178 bug pivotchart pie selected selected reverse
VisActor · Nov 14, 2024 · e8e56ca · e8e56ca
2 parents e468f4b + 475cb49
commit e8e56ca
Show file tree

Hide file tree

Showing 4 changed files with 34 additions and 3 deletions.
diff --git a/.../@visactor/vtable/2178-bug-pivotchart-pie-selected-selected_reverse_2024-11-11-08-51.json b/.../@visactor/vtable/2178-bug-pivotchart-pie-selected-selected_reverse_2024-11-11-08-51.json
@@ -0,0 +1,11 @@
+{
+  "changes": [
+    {
+      "comment": "fix: pivot chart pie type selected state not work #2178\n\n",
+      "type": "none",
+      "packageName": "@visactor/vtable"
+    }
+  ],
+  "packageName": "@visactor/vtable",
+  "email": "[email protected]"
+}
diff --git a/.../@visactor/vtable/2178-bug-pivotchart-pie-selected-selected_reverse_2024-11-11-10-40.json b/.../@visactor/vtable/2178-bug-pivotchart-pie-selected-selected_reverse_2024-11-11-10-40.json
@@ -0,0 +1,11 @@
+{
+  "changes": [
+    {
+      "comment": "fix: rose pivotchart click legend then hover chart render error #2209\n\n",
+      "type": "none",
+      "packageName": "@visactor/vtable"
+    }
+  ],
+  "packageName": "@visactor/vtable",
+  "email": "[email protected]"
+}
diff --git a/packages/vtable/src/PivotChart.ts b/packages/vtable/src/PivotChart.ts
@@ -1181,6 +1181,12 @@ export class PivotChart extends BaseTable implements PivotChartAPI {
         delete spec.area.state.selected;
         delete spec.area.state.selected_reverse;
       }
+      if (spec.pie?.state?.selected) {
+        spec.pie.state.vtable_selected = spec.pie.state.selected;
+        spec.pie.state.vtable_selected_reverse = spec.pie.state.selected_reverse;
+        delete spec.pie.state.selected;
+        delete spec.pie.state.selected_reverse;
+      }
     };
     this.internalProps.indicators?.forEach((indicator: string | IIndicator) => {
       if ((indicator as IChartColumnIndicator).chartSpec) {

diff --git a/packages/vtable/src/scenegraph/refresh-node/update-chart.ts b/packages/vtable/src/scenegraph/refresh-node/update-chart.ts
@@ -297,10 +297,13 @@ export function updateChartData(scenegraph: Scenegraph) {
       cellNode.children.forEach((node: Chart) => {
         if ((node as any).type === 'chart') {
           node.updateData(table.getCellValue(col, row));
-          const newAxes = table.internalProps.layoutMap.getChartAxes(col, row);
-          node.setAttribute('axes', newAxes);
           const chartSpec = node.attribute.spec;
-          chartSpec.axes = newAxes;
+          const chartType = chartSpec.type;
+          if (chartType !== 'gauge' && chartType !== 'rose' && chartType !== 'radar') {
+            const newAxes = table.internalProps.layoutMap.getChartAxes(col, row);
+            node.setAttribute('axes', newAxes);
+            chartSpec.axes = newAxes;
+          }
           node.setAttribute('spec', chartSpec);
           // node.addUpdateBoundTag();
         }