Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UX improvements #268

Open
5 of 6 tasks
tjohnson-scottlogic opened this issue Jul 30, 2024 · 7 comments
Open
5 of 6 tasks

UX improvements #268

tjohnson-scottlogic opened this issue Jul 30, 2024 · 7 comments
Assignees

Comments

@tjohnson-scottlogic
Copy link
Collaborator

tjohnson-scottlogic commented Jul 30, 2024

Various UI improvements as shown below:

Image

Note that the combination of the date picker and the time range is covered by a separate ticket, #292

Acceptance Criteria

  • Summary page - Column width has been reduced (to make the table a bit tighter)
  • City view - The "City" breadcrumb link is not underlined on hover
  • City view - The "City" breadcrumb link has a fixed width so the city name is not shoved across when hovered
  • Graphs - plot points smaller and line thicker to make them sleeker and easier on the eyes. E.g both to 2px.
  • Update the date picker button to say 'Update'
  • On select make the date window dropdown appear in the same colours as the date picker (i.e. dark mode)

Test Checklist:

  • Test Analysis & discussion with developer
  • Write Test Cases / Charters
  • Test Cases & Charters Reviewed
  • Test Cases Executed
  • Test Charters Executed
  • Regression tests executed
  • [ ] Automation Tests added to project
@lincent
Copy link
Collaborator

lincent commented Aug 23, 2024

Graph plot points - PR-321

@mwalker-scottlogic
Copy link
Collaborator

mwalker-scottlogic commented Sep 11, 2024

Test Analysis:

  • As the coding phase is complete, the testing of this is limited in scope to manual testing and to passing regression testing

Manual testing required:

  • check the below points on the page on chrome on a 1920 x 1200 monitor

    • Summary Page:

      • Verify all columns of table fit on the screen in full on a 1920 x 1200 monitor ✅
    • City Page:

      • Verify on hover the "City" breadcrumb link is not underlined ✅
      • Verify on hover the "City" breadcrumb link does not move the the city name across ✅
      • Check each graph - plot points smaller and line thicker to make them sleeker and easier on the eyes ✅ (see pr for evidence
      • Check the date picker confirm button says 'Update' ✅
      • Verify the forecast window dropdown is the same colour as the date dropdown ✅
  • Charter 31: Explore the UI with different screen sizes browsers to discover UX surprises

@mnyamunda-scottlogic
Copy link
Collaborator

Finding
The grid column is set to have less width so the user doesn't have to scroll to see the rest of the pollutants.
In a rare case sometimes the grid opens like so: Image
Meaning the user would have to manually resize the columns to fully see the contents.

@mwalker-scottlogic
Copy link
Collaborator

Finding

AC 'City view - The "City" breadcrumb link has a fixed width so the city name is not shoved across when hovered'

This seems to be working however I have found a small issue with it if the city name is pushed over two lines (eg, go to city page for Rio de Janiro and make the window small until the name splits), then on breadcrumb hover it does get pushed.

Image

@mwalker-scottlogic
Copy link
Collaborator

The grid column is set to have less width so the user doesn't have to scroll to see the rest of the pollutants.
In a rare case sometimes the grid opens like so:

Raised ticket:
https://github.com/orgs/ECMWFCode4Earth/projects/12/views/6?pane=issue&itemId=79693758

@mwalker-scottlogic
Copy link
Collaborator

AC 'City view - The "City" breadcrumb link has a fixed width so the city name is not shoved across when hovered'

This seems to be working however I have found a small issue with it if the city name is pushed over two lines (eg, go to city page for Rio de Janiro and make the window small until the name splits), then on breadcrumb hover it does get pushed.

Raised ticket:
https://github.com/orgs/ECMWFCode4Earth/projects/12/views/6?pane=issue&itemId=79694252

@mwalker-scottlogic
Copy link
Collaborator

Closing ticket as a result

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants