When viewing the pie chart example using the Highcharts library, the chart is broken.

This appears to be that the library does not like to have multiple series when displaying a pie chart.

To resolve this, the number of series in the example should be limited.

CommentFileSizeAuthor
#2 Screen Shot 2018-02-12 at 11.10.40.png46.87 KBJohn Cook
#2 2944095-highcharts_piechart_example-2.patch1.6 KBJohn Cook
Screen Shot 2018-02-12 at 11.06.00.png27.61 KBJohn Cook
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

John Cook created an issue. See original summary.

John Cook’s picture

John Cook CreditAttribution: John Cook at Creode commented
FileSize
2944095-highcharts_piechart_example-2.patch1.6 KB
Screen Shot 2018-02-12 at 11.10.40.png46.87 KB

I created a patch that limits the number of series when a pie chart is used.

A switch statement was used so it would be easier to limit other graph types if necessary.

After applying the patch, the graph displays properly:

John Cook’s picture

John Cook CreditAttribution: John Cook at Creode commented
Status: Active » Needs review
andileco’s picture

andileco CreditAttribution: andileco at JSI Research & Training Institute, Inc. (JSI) commented
Status: Needs review » Fixed

Thanks for the catch. This helped me realize that "Donut" wasn't being included on that default settings configuration page.

  • andileco committed 4f9a33e on 8.x-3.x 
    Issue #2944095 by John Cook: The Pie chart example does not work with...
andileco’s picture

andileco CreditAttribution: andileco at JSI Research & Training Institute, Inc. (JSI) commented
Status: Fixed » Closed (fixed)