-
Notifications
You must be signed in to change notification settings - Fork 6
Open
Description
- Hover/tooltip is done as well as it can be for this version, I think. There is an optimization to get rid of the bsearch, but I'll think about that more and it is probably easier for me to do it than to ask you.
- The x-axis labels are not right, we should probably step away from treating this as time series. However, when I tried other stuff a few weeks ago, the whole thing went haywire. That would be one point where I could use your help.
- The other one is that ChartJS does not have built-in zoom and pan, and we will have to engineer something like click and drag for it, because it would be a shame to not have that.
- There is also a setViewport function in visualization.js that doesn't have logic for chartJS, that needs filling in. This seemed relevant: https://stackoverflow.com/questions/43905749/chart-js-linechart-how-to-show-only-part-of-dataset-and-support-horizontal-scrol
- If we want to get hover useful in live display before they ship version 3.0, we would have to fix the bug ourselves and host our own version.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels