search

nhs-r-community / NHSRplotthedots

An SPC package to support NHSE/I 'Making Data Count' programme
https://nhs-r-community.github.io/NHSRplotthedots/
Other
48 stars 23 forks source link

plotly hover tooltips need improving #178

Open tomjemmett opened 1 year ago

tomjemmett commented 1 year ago

current plotly implementation uses default tooltips, these could do with being improved

ThomUK commented 1 year ago

Just some screen grabs from when I was reviewing Current UPL tooltip (needs rounding, and better label): image

Current data tooltip (might need rounding for some datasets, point_type would be better as "Variation Type", and removing the snake case text would be great): image

Similar points to above for the legend: image

tomjemmett commented 1 year ago

separate issue for legend in #177 🙂

tomjemmett commented 1 year ago

182 is an attempt to improve this, but not resolve. I've basically hacked it by string replacing on the items to remove the additional text, then used a unified x-axis hovermode to show all traces at once

We may be somewhat limited by using plotly::ggplotly, and to tackle this completely we may need to create it in {plotly} from scratch

image

joekeane7 commented 7 months ago

Is there any work ongoing around improving the tooltips? I think it would be good to see the cause of the dot colour like shift above the mean for example when you hover over a dot.

image

ThomUK commented 7 months ago

I don't think anyone is working on it, so feel free to self-assign it if you'd like to...