clausherther commented on PR #19038: URL: https://github.com/apache/superset/pull/19038#issuecomment-1105241783
Hi @stephenLYZ, the use case you're describing makes sense - keeping chart dimension colors consistent. This was already possible via `label_colors`. The issue we're running into is that on a dashboard with a number of different charts, new charts suddenly get assigned random colors outside of Airbnb's color palette. This is unexpected and undesirable for our users and will ultimately hurt adoption of our dashboards. What we're observing is also that charts that do *not* have dimensional cuts (i.e. simple single metric time-series charts without group-bys or multiple series), which in the past have all been assigned the first value in the color palette, now get assigned a new, non-palette, color value, despite having no dimensional cuts. Would it be possible to make the behavior you've implemented optional at the dashboard level? E.g. "Cycle through dashboard theme colors: Yes/No" with "No" describing the behavior you've implemented? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
