| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
There was a problem hiding this comment.
As we discussed, there's an issue with implementing the split: true logic that makes the split hover info not show up. Could you look into that?
Sorry, something went wrong.
Co-authored-by: Cameron DeCoster <cameron.decoster@gmail.com>
Co-authored-by: Cameron DeCoster <cameron.decoster@gmail.com>
|
@camdecoster The bug you found with split: true should be fixed now. Ready for another look. I also added a Jasmine test which tests split: true and hovertemplate together. |
Sorry, something went wrong.
There was a problem hiding this comment.
Looks good. Could you please add a draftlog before merging?
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Closes #3437
This PR adds a hovertemplate attribute for ohlc and candlestick, allowing users to define a custom hover template for those traces.
Notes
The first commit of this PR runs the biome formatter which introduces some noise into the diff. Use this link to see only the meaningful changes.
This PR also adds the hovertemplatefallback attribute for the two traces in question
hovertemplate is ignored when hoverlabel.split is set to true. (Open to suggestions here but I couldn't see an obvious way to make split: true work with the hovertemplate API.)
Two Jasmine tests added
This completes the list in #3437.
Thanks to @chriddyp for putting this on my radar!
To test