Uh oh!
There was an error while loading. Please reload this page.
Add info message for area trace opacity - #198
Conversation
| const cannotSetTraceOpacity = | ||
| fullContainer.type === 'scatter' && | ||
| (fullContainer.fill !== 'none' || fullContainer.fill !== 'toself'); | ||
There was a problem hiding this comment.
@alexcjohnson is cannotSetTraceOpacity correct?
I feel like since there's an accordion menu for each trace in the editor, the rest of this logic doesn't apply:
https://github.com/plotly/plotly.js/blob/master/src/traces/scatter/clean_data.js#L24
There was a problem hiding this comment.
if we're looking at the trace from fullData, we should just be able to use the results of scatter/clean_data, ie fullContainer.type === 'scatter' && fullContainer.opacity === undefined, right? Because the multi-trace aspect of it is really important for alleviating confusion and you're right that you don't want to try and reconstruct that here.
28735ff to
f33b2abCompareVeraZab
commented
Jan 5, 2018
@bpostlethwaite@alexcjohnson can I merge? |
alexcjohnson
commented
Jan 5, 2018
want to rebase while I look at it again? |
8e18fe2 to
e7e9f03Compare
closes: #70
