-
Notifications
You must be signed in to change notification settings - Fork 644
better UI for 7-day moving average graphs #567
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
Comments
I agree that the dots look strange at first glance, though I think the current legend suffices. And I think adding a toggle might be too much, I'd suggest just removing the dots and leaving the ability to hover over a point on the graph in order to see the raw data for that period. |
I like the dots a lot. |
When hovering the version numbers, the dots for that version get highlighted but not the line, that looks like a bug. I suggest just using a checkbox to toggle the 7-day average off. I think using the average for smoothing is fine, but going half-way and showing both is just confusing. The dots just look like noise right now. |
this is incredibly hard to read. colors for the different versions dots are barely different from each other, and because they aren't connected with lines, its super hard to tell what dot goes with what version. how about put 7 day average into a different graph, or provide a toggle to make it visible for people who care about it? |
I would love pull requests for changing any of these things! |
I don't think we're going to do this. |
The moving average lines added in #552 initially confused me -- I thought the output was broken. There is no UI describing why the graph has both lines and dots. I suggest either some explanatory text in the graph legend or a toggle to switch between fine-grained and averaged plotting.
The text was updated successfully, but these errors were encountered: