feat(frontend): add option to unstack downloads per version #1101
Conversation
|
this is intentional behaviour, and is common across other systems already (ie crates.io). a stacked graph makes sense for an aggregation set of data across different points, so this seems the right usecase |
|
ah dangit |
There's an option on crates.io to unstack the graph. Would you be ok with me changing this to add the dropdown option for stacked/unstacked here as well? 
|
|
@tylersayshi yes that sounds good |
tylersayshi
changed the title
| out = new Date(Date.UTC( | ||
| date.getUTCFullYear(), | ||
| date.getUTCMonth(), | ||
| date.getUTCDate() - date.getUTCDay(), | ||
| )); | ||
| out = new Date( | ||
| Date.UTC( | ||
| date.getUTCFullYear(), | ||
| date.getUTCMonth(), | ||
| date.getUTCDate() - date.getUTCDay(), | ||
| ), | ||
| ); |
There was a problem hiding this comment.
Why might this change? I ran deno fmt, is the style not guarenteed to stay the same?
| const isDarkMode = document.documentElement.classList.contains( | ||
| "dark", | ||
| ); | ||
| const newAggregationPeriod = e.currentTarget | ||
| .value as AggregationPeriod; |
There was a problem hiding this comment.
took the chance to simplify this logic while I was reading it and adding the other select.
only updateSeries is needed for this
adds option to view stacked and unstacked downloads (keeping current defaults). Replicating the downloads chart for crates.io
stacked:
unstacked:
options on small screens:
