fix(mixins): Check for chartjs instance before rendering chart

Closes #288
apertureless · Jan 12, 2018 · 39ff839 · 39ff839
1 parent 0506b4e
commit 39ff839
Show file tree

Hide file tree

Showing 3 changed files with 552 additions and 484 deletions.
diff --git a/src/mixins/reactiveData.js b/src/mixins/reactiveData.js
@@ -63,6 +63,9 @@ module.exports = {
             this.renderChart(this.chartData, this.options)
           }
         } else {
+          if (this.$data._chart) {
+            this.$data._chart.destroy()
+          }
           this.renderChart(this.chartData, this.options)
         }
       }

diff --git a/src/mixins/reactiveProp.js b/src/mixins/reactiveProp.js
@@ -63,6 +63,9 @@ module.exports = {
             this.renderChart(this.chartData, this.options)
           }
         } else {
+          if (this.$data._chart) {
+            this.$data._chart.destroy()
+          }
           this.renderChart(this.chartData, this.options)
         }
       }