-
Notifications
You must be signed in to change notification settings - Fork 41
Add sparklines or barchart component #1434
Comments
@shawnbot What's the status on this one? I know you've got a WIP on this. Can we merge it into the main state profile branch and use the spark to implement @ericronne 's design? |
I'd love @ericronne's input on #1448, then we can close this out when it merges. |
@ericronne @shawnbot Did y'all chat on this? Can we merge? Thx! |
Should we turn this into the bar chart design issue? |
Sure! To be clear, I think you mean: turning the line charts into the latest design on the bar charts, right @shawnbot ? |
Yep, @meiqimichelle! |
@shawnbot This may not be bad (at least for now), especially if these charts with various numbers of columns don't commingle. |
Yep, good call @ericronne. The outlier here might be whichever of the all lands production commodities has data back to the 1970s, but in that case we can either constrain the year range the data side (to fit the narrowest common window), or explicitly limit all of the related bar charts to the same year range so it doesn't look weird (and so that it's less confusing). cc @meiqimichelle |
👍 On Wed, Jun 29, 2016 at 2:07 PM, Eric Ronne [email protected]
Judy Wilson [email protected] [email protected] |
Refs
Design: #1348
Code: #1431
Like this!
The text was updated successfully, but these errors were encountered: