Skip to content

Commit

Permalink
Allow registering plugins in individual chart instances
Browse files Browse the repository at this point in the history
Fixes #1456
  • Loading branch information
santam85 committed Nov 23, 2022
1 parent 9dba839 commit 22b626b
Showing 1 changed file with 12 additions and 5 deletions.
17 changes: 12 additions & 5 deletions projects/ng2-charts/src/lib/base-chart.directive.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,14 @@ import {
Output,
SimpleChanges,
} from '@angular/core';
import { Chart, ChartComponentLike, ChartConfiguration, ChartEvent, ChartType, DefaultDataPoint } from 'chart.js';
import {
Chart,
ChartConfiguration,
ChartEvent,
ChartType,
DefaultDataPoint,
Plugin
} from 'chart.js';

import { ThemeService } from './theme.service';
import { Subscription } from 'rxjs';
Expand All @@ -30,7 +37,7 @@ export class BaseChartDirective<TType extends ChartType = ChartType,
@Input() public legend?: boolean;
@Input() public data?: ChartConfiguration<TType, TData, TLabel>['data'];
@Input() public options: ChartConfiguration<TType, TData, TLabel>['options'];
@Input() public plugins: ChartComponentLike[] = [];
@Input() public plugins: Plugin<TType>[] = [];

@Input() public labels?: ChartConfiguration<TType, TData, TLabel>['data']['labels'];
@Input() public datasets?: ChartConfiguration<TType, TData, TLabel>['data']['datasets'];
Expand Down Expand Up @@ -62,6 +69,7 @@ export class BaseChartDirective<TType extends ChartType = ChartType,
} else {
const config = this.getChartConfiguration();

// Using assign to avoid changing the original object reference
if (this.chart) {
Object.assign(this.chart.config.data, config.data);
if (this.chart.config.plugins) {
Expand Down Expand Up @@ -89,8 +97,6 @@ export class BaseChartDirective<TType extends ChartType = ChartType,
this.chart.destroy();
}

Chart.register(...this.plugins);

return this.zone.runOutsideAngular(() => this.chart = new Chart(this.ctx, this.getChartConfiguration()));
}

Expand Down Expand Up @@ -158,7 +164,8 @@ export class BaseChartDirective<TType extends ChartType = ChartType,
return {
type: this.type,
data: this.getChartData(),
options: this.getChartOptions()
options: this.getChartOptions(),
plugins: this.plugins
};
}

Expand Down

0 comments on commit 22b626b

Please sign in to comment.