search

victronenergy / gui-v2

Other
24 stars 9 forks source link

Solar Yield drilldown History page Table View is missing data rows #1305

Open chriadam opened 2 months ago

chriadam commented 2 months ago

Select range to 30 days, the Table View is empty. To get per-day data, you have to go to Chart view and then click on a specific day's bar. That specific day data should be shown as rows in the Table View (which should be scrollable).

I don't believe this should block 1.0.0 release, setting as medium_prio for now, but will ask @Serj and @jepefe for input.

chriadam commented 2 months ago

Also, minor UI fix: the per-day bars in the chart view should be "outline style" rather than "filled style" to hint to the user that they are clickable (i.e. match the visual style to overview widgets with drilldowns).

blammit commented 2 months ago

I'm wondering, if there are rows of per-day data, what would they provide? Is it just the Yield, Max Voltage and Max Power, or also the other data like the Battery, Charge Time and Errors?

Also this will need some rearranging of the design, since the Table View currently shows a summary of the last X days, rather than per-day data, so if we change this it would need to be clear about which parts provide the summary and which parts provide the per-day data.