Skip to content

Commit

Permalink
Add support for polar charts
Browse files Browse the repository at this point in the history
- As per [documentation](https://www.highcharts.com/docs/chart-and-series-types/polar-chart):

> Polar charts, also known as radar charts, require the highcharts-more.js file.

I was also tinkering with the idea to just merge the if to the "waterfall" case below as it's essentially the same, but then decided against as it might cause issues in the future.
  • Loading branch information
MichalBryxi authored Sep 3, 2024
1 parent 54aee88 commit 7995a3e
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions ember-highcharts/src/components/high-charts.ts
Original file line number Diff line number Diff line change
Expand Up @@ -256,6 +256,11 @@ export default class HighCharts<
SolidGauge.default(Highcharts);
}

if (this.args.chartOptions?.chart?.polar === true) {
const More = await waitForPromise(import('highcharts/highcharts-more'));
More.default(Highcharts);
}

if (this.args.chartOptions?.chart?.type === 'waterfall') {
const More = await waitForPromise(import('highcharts/highcharts-more'));
More.default(Highcharts);
Expand Down

0 comments on commit 7995a3e

Please sign in to comment.