DFE-Digital / schools-experience

The Department for Education's Get Schools Experience Service which allows you to find out more about teaching by visiting schools.
https://dfe-digital.github.io/schools-experience/
MIT License
9 stars 4 forks source link

Improve button groupings (first tranche) #3157

Closed MylesJarvis closed 5 months ago

MylesJarvis commented 5 months ago

Trello card

https://trello.com/c/QHKZbml7

Context

Changes proposed in this pull request (changes in comments below)

Guidance to review

Attendance page (you will need to login as a school with attendances to confirm (Holland Park) and one without (Outwood Academy Acklam)):

Toggle page on/off:

Edit profile page (use any onboarded school)

Add dates (use any school with 'fixed dates'):

Onboarding flow (use any non-onboarded school and complete their profile details)

For the moment the main focus can be grouping - I can do separate PRs for changing buttons -> links if needed. For example, the return to dashboard buttons.

MylesJarvis commented 5 months ago

Splitting each change into comments to make it easier to review:

Before - turn profile on/off

image

After - turn profile on/off

image
MylesJarvis commented 5 months ago

Before

image

After

image

Note: I'll do a separate PR for the duplication of 'showing the only booking' line.

MylesJarvis commented 5 months ago

Before

image

After

image
MylesJarvis commented 5 months ago

Before

image

After

image
MylesJarvis commented 5 months ago

Before

image

After

image
MylesJarvis commented 5 months ago

Before image

After

image

gemmadallmandfe commented 5 months ago

LGTM other than suggested content tweak

github-actions[bot] commented 5 months ago

Review app deployed to https://get-school-experience-review-pr-3157.test.teacherservices.cloud :white_check_mark: DfE sign in route obtained: https://get-school-experience-review-pr-4.test.teacherservices.cloud