-
Notifications
You must be signed in to change notification settings - Fork 17
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
add histogram back #749
add histogram back #749
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should have this work irrespective of #748 (the whitespace)
I don't think that is realistic. To do that we need all the charts be the exact same height, which is the issue with the grid. |
<WidgetDiv> | ||
<WidgetTitle | ||
title="Headway histogram" | ||
subtitle="Time spent at station" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This isn't correct, unless you mean "time rider spends at station waiting for train"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
whoops. Good catch.
I don't love using "histogram" in the visible display text. Maybe "distribution"? |
Motivation
Forgot to add the headways histogram back.
Changes
Adds headways histogram back.
Removing the grid in #748 which makes the whitespace at the botom go away.
Testing Instructions