Conversation
|
This looks OK, but it wouldn't be that hard to have the bars adapt to the background color: on a dark background make them white, on a light background make them black. Just give them a class but no fill or stroke, and set it explicitly (to |
Good call and I could definitely try that. |
|
True. But if it’s not over the background color it’s likely to be over a mix of multiple colors… in which case short of filling around the logo with a solid color (@angeladaodao is that what you were suggesting?) there’s really nothing we can do to make it robustly visible. |
|
That is correct |
Co-authored-by: Alex Johnson <alex@plot.ly>
alexcjohnson
left a comment
There was a problem hiding this comment.
💃 (BTW when looking at the changelog to verify the correct form I notice a few recent ones missing the # - perhaps something to fix while prepping for the next release)
demo
@plotly/plotly_js
cc: @angeladaodao
Please note that we use
#777(with a white border line) instead of plain black and white bars i.e. to handle various background colors in plotly.js graphs.Also gradient in the original logo is replaced by simple colors for points.