Mateusz Grotek
Mateusz Grotek
@dependabot rebase
A separate PR https://github.com/alphagov/frontend/pull/4058 with those changes has already been merged.
Closing because of the above.
This PR is waiting for the final confirmation of the icon sizes and one of the translations.
I've cropped the images.
> Thanks Mat, > > I've requested some changes, hopefully the comments makes sense? > > I'd like to split this PR to make it easier for us to get...
@hannako I've checked and `ContentChange` also uses `taxon_tree`. This is added by [this code](https://github.com/alphagov/email-alert-service/blob/c2bbc08a16af48cd4b5f4fd8dcc57ca533fdbee5/email_alert_service/models/document_links.rb#L13) in `email-alert-service`. This simplified things a bit. If a test already uses `taxon_tree`, I've used `document_collections`...
> Thanks for this. It does feel like we should be able to set this with a global option in the network bit, I'd want us to rule that out...
> I've had a quick look and have one thought on the updated breadcrumbs, using the page below as an example: > > [www.gov.uk/government/organisations/government-actuarys-department/about](http://www.gov.uk/government/organisations/government-actuarys-department/about) > > The breadcrumbs are updated...
Or maybe we could use something like `Home > Organisations > Government Actuary's Department` as in the Government Actuary's Department's page the breadcrumbs are `Home > Organisations`?