Add support for (hover) 'text' in mesh3d traces#2327
Conversation
|
Not sure how much you want to let this PR expand, but here's a definitely-a-bug that popped up as I was trying out your PR: Also it looks like |
|
@alexcjohnson good catch. Thanks. I think I fixed them all in 8ae2d0e by overriding the |
|
8ae2d0e looks great, thanks for the tests 🔒 |
Oops. I pushed too soon. That's in d20bdc1 |
src/traces/surface/attributes.js
Outdated
| role: 'info', | ||
| dflt: '', | ||
| arrayOk: true, | ||
| editType: 'calc', |
There was a problem hiding this comment.
Oh this is in an overrideAll so you don't need editType - which I think applies to a few more of the additions in this PR too.
There was a problem hiding this comment.
(I see you 🔪 some editType lines, just not the one I made the comment directly on 😅 )
|
Nice - slowly getting all these trace types up to par with each other. 💃 |
Call this a bug fix or a new feature, this PR closes #2242