Now, this module have hard-coded default colors for all libraries.
I think, that settings from chart configuration page should be loaded to module.
In my case I'm using tablefield_charts and when i want to change colors settings, it was impossible.

Patch attached

CommentFileSizeAuthor
#2 Before patch.png7.03 KBkrina.addweb
#2 After Patch.png4.86 KBkrina.addweb
default-colors-from-settings.patch495 byteslamp5
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lamp5 created an issue. See original summary.

krina.addweb’s picture

krina.addweb CreditAttribution: krina.addweb at AddWeb Solution Pvt. Ltd. commented
Status: Needs review » Reviewed & tested by the community
FileSize
After Patch.png4.86 KB
Before patch.png7.03 KB

@lamp5, Thanks for providing updated Patch it works well as per required changes. I've checked your patch using simplytest.me & Attached the Results for the same.

clemens.tolboom’s picture

clemens.tolboom
Primary language Dutch
Location Groningen, 🇳🇱/🇪🇺
CreditAttribution: clemens.tolboom as a volunteer commented
Issue summary: View changes 
+++ b/charts.module
@@ -359,10 +359,9 @@ function charts_charts_type_info() {
+    ¶

Whitespace

  • andileco committed 54eff6f on 7.x-2.x authored by lamp5 
    Issue #2860723 by lamp5, krina.addweb: Default colors from configuration...
andileco’s picture

andileco CreditAttribution: andileco at JSI Research & Training Institute, Inc. (JSI) commented
Status: Reviewed & tested by the community » Fixed
andileco’s picture

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