- Issue created by @Liam Morland
- Merge request !88Issue #3442641: Add colors for each segment in pie and donut charts β (Merged) created by Liam Morland
- Status changed to Needs review
about 1 year ago 8:57pm 22 April 2024 - π¨π¦Canada Liam Morland Ontario, CA π¨π¦
I was able to get the colors working by adding a patch to
charts_billboard
. The patch deals with#grouping_colors
, which is currently not used bycharts_billboard
. I note thatcharts_highcharts
does have code that uses#grouping_colors
. - π¨π¦Canada Liam Morland Ontario, CA π¨π¦
Current merge request contents as a patch.
- π¨π¦Canada Liam Morland Ontario, CA π¨π¦
This patch handles non-existent #grouping_colors. Merge request updated.
- πΊπΈUnited States andileco
Thank you for your patch! If you wanted to, you could add that identical code in the C3.js class.
Billboard.js and C3.js are a little neglected within the Charts module just because I end up using Highcharts for my work. I appreciate you identifying, posting, and patching!
- π¨π¦Canada Liam Morland Ontario, CA π¨π¦
OK, I added identical code to
charts_c3
in the merge request. I have not tested it since I don't use it. -
andileco β
committed 61bf28c6 on 5.0.x authored by
Liam Morland β
Issue #3442641 by Liam Morland: Chart segments are all the same color in...
-
andileco β
committed 61bf28c6 on 5.0.x authored by
Liam Morland β
- Status changed to Fixed
about 1 year ago 5:42pm 24 April 2024 - πΊπΈUnited States andileco
I tested and confirmed it works - thanks again!
Automatically closed - issue fixed for 2 weeks with no activity.