Update charts.ts

Refresh graph on label/option/chartType/color/legend change
MumblesNZ · Jan 16, 2019 · bdcaedb · bdcaedb
1 parent 602742b
commit bdcaedb
Showing 1 changed file with 16 additions and 12 deletions.
diff --git a/src/charts/charts.ts b/src/charts/charts.ts
@@ -64,18 +64,22 @@ export class BaseChartDirective implements OnDestroy, OnChanges, OnInit {
 
   public ngOnChanges(changes: SimpleChanges): void {
     if (this.initFlag) {
-      // Check if the changes are in the data or datasets
-      if (changes.hasOwnProperty('data') || changes.hasOwnProperty('datasets')) {
-        if (changes['data']) {
-          this.updateChartData(changes['data'].currentValue);
-        } else {
-          this.updateChartData(changes['datasets'].currentValue);
-        }
-
-        this.chart.update();
-      } else {
-      // otherwise rebuild the chart
-        this.refresh();
+      if (changes.hasOwnProperty('labels') || changes.hasOwnProperty('options') || changes.hasOwnProperty('chartType') || changes.hasOwnProperty('colors') || changes.hasOwnProperty('legend')) {
+          this.refresh();
+      }
+      else if (changes.hasOwnProperty('data') || changes.hasOwnProperty('datasets')) {
+          // Check if the changes are in the data or datasets
+          if (changes['data']) {
+              this.updateChartData(changes['data'].currentValue);
+          }
+          else {
+              this.updateChartData(changes['datasets'].currentValue);
+          }
+          this.chart.update();
+      }
+      else {
+          // otherwise rebuild the chart
+          this.refresh();
       }
     }
   }