-
Notifications
You must be signed in to change notification settings - Fork 74
Group tooltips for line chart #1336
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
NOTE: I will add tests for this once #1331 is merged. |
Co-authored-by: Joe Pavitt <99246719+joepavitt@users.noreply.github.com>
|
Merged your correction to the Addressed you comment about |
|
Tests updated/added to cover changes for scatter and bar. All passing bar one of the control spec tests (usual suspect) (really would love to get that reliable!) |
@Steve-Mcl - @cstns and I have collectively spent days trying - we've given up. |
|
Works great - thanks @Steve-Mcl |
Description
Groups tooltips on
linetype charts using the interactions optionsNOTES:
this.chart.updateinto achartUpdatemethod so that we can debounce the multiple calls generated by property updates (the props arewatched individually)chartobject to thewindowobject so that during debugging, I can see the actual config and data applied to the chart. This was super useful. So instead of deleting it, I let it in commented with a note for future devs to take advantage of. fee free to remove it.Related Issue(s)
#1254
Checklist
flowforge.yml?FlowFuse/helmto update ConfigMap TemplateFlowFuse/CloudProjectto update values for Staging/ProductionLabels
area:migrationlabel