Setting default colors

[cixzhang]

Could we have a way to override the default colors for plotlyjs?

I'm willing to work on this feature.

[etpinard]

Can you be more specific?

You would like to make the default trace-to-trace color sequence customizable?

[cixzhang]

@etpinard Yup! Exactly! I'd like to be able to set a specific color scheme for all chart traces to use.

[etpinard]

Ok. I'm thinking this should be a config option e.g. traceColorDefaults.

[cixzhang]

That sounds good to me! I'll take a stab at it tonight.

[cixzhang]

Following the PR guidelines, I made a PR here in my own fork: cixzhang#1.

Please have a look when you have time!

[alexcjohnson]

@cixzhang Great PR! @etpinard and I had a discussion about this offline though, and we concluded it actually should be a top-level layout attribute, not a config option. The advantage of this is that the result is portable (on plot.ly for example data and layout get saved with the plot but config is context-dependent), and easier to change (via relayout / update) - which can be useful for example if you want to try out a bunch of different themes / templates for your plot (cc @bpostlethwaite )

We're thinking layout.colorway - sound reasonable?

[cixzhang]

Thanks for the feedback! That makes sense to me. I'll take a look at implementing it as a layout attribute this evening.

[cixzhang]

I've created a new PR with layout.colorway on my fork here: cixzhang#2

[cixzhang]

I have updated my PR here: cixzhang#2. Let me know if there's anything else I should do. :)

[bpostlethwaite]

Thanks @cixzhang we appreciate the PR and will get to it ASAP. Unfortunately that may be a few days as our core plotly.js team is either on vacation or dealing with unforeseen events. Everything should be back to normal next week. Thanks again.

[etpinard]

Done in #2156