Add breadcrumbs to pages without them

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

Add breadcrumbs to pages without them #3168

Closed MylesJarvis closed 1 month ago

MylesJarvis commented 2 months ago

Trello card

https://trello.com/c/gapHzCUD

Context

Some pages lack breadcrumbs
Some pages lack backlinks

Changes proposed in this pull request

Add breadcrumbs to the toggle profile page
Add breadcrumbs to choose how dates are displayed page
Add backlinks to contact us/request access to another organisation/close placement dates page

Guidance to review

Do breadcrumbs render on the profile on/off page?
Do breadcrumbs render on the 'choose how dates are displayed' page?
Do the breadcrumbs link to where you expect?
Are backlinks appropriate in the other locations?
Should we use breadcrumbs in the close placement date page?

MylesJarvis commented 2 months ago

Closing a placement date

Before

After

Option 1 - now replaced

Option 2 - currently deployed in preview app

Questions for review: Does this also require a 'return to dashboard' link? Is the back button the appropriate module here, or do we think a fully featured breadcrumb is a better option?

MylesJarvis commented 2 months ago

Turn profile on/off

Before