BiologicalRecordsCentre / UKBMS-online

Issue tracking for UKBMS online recording site
2 stars 0 forks source link

Changes to My Sites page on new website #170

Closed MeganLowe closed 3 years ago

MeganLowe commented 3 years ago

On the My Sites page of new website:

DavidRoy commented 3 years ago

@BirenRathod can you make these changes to https://test-ukbms8.pantheonsite.io/my-sites

BirenRathod commented 3 years ago

@MeganLowe I have made below amendments.

Gary-van-Breda commented 3 years ago

@BirenRathod @MeganLowe : the default Site Type should be remembered across the different reporting pages, so if the user chooses site type "WCBS-BBS" in the Annual summary, it should be the default the next time the My Sites is brought up.

BirenRathod commented 3 years ago

@Gary-van-Breda yes, I missed the remembering part. So no change requires.

MeganLowe commented 3 years ago

@Gary-van-Breda : I find that the My Sites page defaults to Transect on first open. It may change after you've been to the Annual Summary page (I haven't tested this) but most people would go to My Sites first. The trouble is we often find that WCBS recorders struggle to find their squares if the drop down box defaults to transect and then they create a new transect instead (not what we want). If it's too tricky to change the default then I will just write some clearer instructions on the page - not a problem.

MeganLowe commented 3 years ago

@Gary-van-Breda @BirenRathod : sorry, also just noticed 'Report Parameters' text is still present on the My Sites page.

Gary-van-Breda commented 3 years ago

@MeganLowe : I've changed the default default (sic) to "please select" on "My sites" only.

BirenRathod commented 3 years ago

@MeganLowe I implemented the style. 'Report Parameters' text is hidden now.

MeganLowe commented 3 years ago

@Gary-van-Breda thanks. Is it possible to make the page automatically show all the sites assigned to you (as if the ‘Run Report’ button had already been pressed)? Then the filter just provides the user with the option to limit the list if they want to. Also, it would be useful if there was an additional column for ‘Site Type’ in the table that is shown. So that when all your sites are showing it’s easy to differentiate between eg. WCBS-BC and WCBS-BBS squares. Is that possible please? Thanks, Megan (and Ian)

Gary-van-Breda commented 3 years ago

@MeganLowe : I've tweaked it so the report now runs automatically with no site type filter applied. @BirenRathod : Can you deploy the current master Branch of the BRC reports (https://github.com/BiologicalRecordsCentre/indicia-reports) into warehouse1? Thanks (@MeganLowe this report update is required so we can get access to the location type for the report grid)

BirenRathod commented 3 years ago

@Gary-van-Breda I have pulled that report now.

Gary-van-Breda commented 3 years ago

@MeganLowe : back to you to check that this is what you want...

MeganLowe commented 3 years ago

@Gary-van-Breda this is fantastic! Just one other small thing, please can you make the space between the 'Run Report' button and the table below ever so slightly bigger?

Gary-van-Breda commented 3 years ago

@BirenRathod : can you do the "make the space between the 'Run Report' button and the table below ever so slightly bigger" ?

BirenRathod commented 3 years ago

@Gary-van-Breda & @MeganLowe I have increased a space now.

MeganLowe commented 3 years ago

Brilliant, looks great thank you.