[Improvement] Order Overview Statistics Sections

[jackpugh2000]

Summary

In the overview pages, the results are aggregated in the below sections. Sometimes it may be beneficial to see the aggregated data at the top of the page, rather than nested below.

Expectation

In the overview page, make the sections order-able (like the dashboard sections that can also be ordered).

[jackpugh2000]

@timdegroot1996

I may need to have a further think on what I was wanting this page/section to provide. I’ll try to write down some notes and comment them here this week.

[timdegroot1996]

@jackpugh2000 I think I already implemented this (fully) in the PR I mentioned to you. I forgot to tell you as there was no issue yet but PR #168 also has this. If you press the "customize layout" in the overview page you can already change the order. Which is saved to your localstorage.

Could you check if that is what you are describing above?

[jackpugh2000]

@timdegroot1996

Ah okay. The customising of the order in the overview page doesn't seem to work for me on that example? Is it for you?

I'm not sure if just changing the order will do exactly what I was wanting, so I'll see if I can better write it up

[timdegroot1996]

@jackpugh2000 This has been released in 1.5.0. Let me know if this ticket can be closed!

[jackpugh2000]

@timdegroot1996 I noticed while looking at this that the top section 'Overview Statistics' doesn't move when you select the up & down icons.

I also think there could be further work to this section, but will raise that in its own issue.

[timdegroot1996]

@jackpugh2000

@timdegroot1996 I noticed while looking at this that the top section 'Overview Statistics' doesn't move when you select the up & down icons.

I looked into this and it does for me. The order does actually change although visually it might not be as clear. Can you double check that this is the case?

I also think there could be further work to this section, but will raise that in its own issue.

Does that mean we can close this issue?

Edit: If it is not I want to clarify what is required and also apply it in PR #182

[jackpugh2000]

@timdegroot1996 I think when I raised this I was testing my local dashboard when running as a server. But I've just checked your example dashboard and it works fine - so I'll double check and get back to you!

[timdegroot1996]

@jackpugh2000 Added in release 1.6.0 (maybe even 1.5.0). Closing this issue as it has been added. If issues arise let me know!