Fix bar marker color array legend style #1289
Conversation
- as box traces don't have arrayOk style attribute at the moment
| ] | ||
| ], | ||
| "layout": { | ||
| "showlegend": true |
There was a problem hiding this comment.
| @@ -172,13 +172,14 @@ function styleBars(d) { | |||
| barpath.exit().remove(); | |||
| barpath.each(function(d) { | |||
| var w = (d.mlw + 1 || markerLine.width + 1) - 1, | |||
There was a problem hiding this comment.
does this need d -> d0 too? care to include line width and color arrays into the same mock?
There was a problem hiding this comment.
Good 👁️
Done in 8e718bf
There was a problem hiding this comment.
for completeness:
and color arrays
ie marker.line.color
| "red", | ||
| "green", | ||
| "blue", | ||
| "yellow" |
There was a problem hiding this comment.
A thing of beauty :)
|
💃 |
|
@etpinard, Im afraid this is occurring again in 2020. Could you take a look please? |
|
@nitramwin can you open a new issue with a codepen showing the problem you've encountered? |
|
@alexcjohnson This is my first time making a codepen - I hope this works. https://codepen.io/nitramwin/pen/mdEoxZo |
Thanks @nitramwin. Tracked in #5285. |
fixes #1212
... bringing legend support for bar maker color arrays on par with scatter.