coloraxes with values for sunburst and treemap

[archmoj]

Shared coloraxes should be implemented for sunburst and treemap.

[etpinard]

Hmm. They seem to work fine already:

https://codepen.io/etpinard/pen/zYxdKRw

[archmoj]

Interesting. So the user have to provide a color array and cannot use values, right?

[etpinard]

Interesting. So the user have to provide a color array and cannot use values, right?

Well, too late for that. You handle the values w/o marker.colors case here:

plotly.js/src/traces/sunburst/calc.js

Lines 203 to 205 in e778644

	if(!hasColors) {
	colors = hasValues ? trace.values : trace._values;
	}

[nicolaskruchten]

Does this really belong in the 1.52 milestone?

[etpinard]

It's done in #4444 - which we'll merge at some point today