Timeseries UI UX Enhancements

LanesGood commented 1 year ago

Todo:

Form controls

[ ] Provide more instructions on timeseries selection and binning/interval
[ ] Require team or user selection before country selection
[x] Add "submit" button - in #708

Table

[x] Add background to table
[ ] Sort columns by total # edits
[x] Sortable column controls

Chart

[x] Add background to chart
[ ] Add dynamic title to chart
[ ] Empty state (this query returned no results) - revise?
[ ] Restrict chart to certain number of users
[ ] Add title to legend

General page UI

[X] Add page header
[x] Confirm layout responsiveness
[X] Move "Export" button to results pane

LanesGood commented 1 year ago

Current progress on styles for timeseries layout:

LanesGood commented 1 year ago

@kamicut @maxgrossman I believe this is ready for review. In the task list above, I'm leaving all "Form Controls" for a different pr/developer. I want to ensure we're on the same page on requiring certain selections.

Chart I don't know that a separate title for the chart, or legend titles or a different empty state text line is necessary. I do think we should restrict the number of data points for a chart, but will also leave that for a different discussion and PR.

Table Sorting Without changing anything related to sorting, sorting seems to have been enabled on this branch. I do think this is a preferred approach but am not sure if it was disabled on the main branch due to performance concerns.

developmentseed / scoreboard