fix: when chart's data changed, clear the append shape. Closed #227.

antvis · Aug 3, 2018 · 9090379 · 9090379
1 parent e79d7eb
commit 9090379
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 0 deletions.
diff --git a/demos/0-interacion-pie-select.html b/demos/0-interacion-pie-select.html
@@ -8,6 +8,7 @@
 <body>
 <div>
   <canvas id="mountNode"></canvas>
+  <button id="changeBtn"> changeData </button>
 </div>
 <script src="./assets/jquery-3.2.1.min.js"></script>
 <script src="../build/f2-all.js"></script>
@@ -85,6 +86,11 @@
 });
 
 chart.render();
+
+var btn = document.querySelector('#changeBtn')
+btn.onclick = function() {
+	chart.changeData([ {  name: 'test1', percent: 0.1, a: '1'}, {  name: 'test2', percent: 0.1, a: '1'} ])
+}
 </script>
 </body>
 </html>
diff --git a/src/interaction/pie-select.js b/src/interaction/pie-select.js
@@ -16,6 +16,19 @@ class PieSelect extends Interaction {
     });
   }
 
+  constructor(cfg, chart) {
+    super(cfg, chart);
+    const self = this;
+    chart.registerPlugins({
+      clearInner() {
+        self.halo && self.halo.remove(true);
+        self.selected = false;
+        self.selectedShape = null;
+        self.lastShape = null;
+      }
+    });
+  }
+
   start(ev) {
     const chart = this.chart;
     if (ev.type === 'tap') {