Skip to content

Conversation

@owenatgov
Copy link
Contributor

@owenatgov owenatgov commented Jun 16, 2025

What’s changed

Applies the brand refresh introduced in GOV.UK Frontend 5.10.0 to tech docs when show_govuk_logo is set to true. If show_govuk_logo is falsey, it will show the black header with blue bottom border.

As part of this, this PR also:

  • updates govuk-frontend to 5.10.2
  • updates the GOV.UK logo svg in the header
  • adds the crown svg to the footer and ties its rendering to show_govuk_logo
  • adds a new test to the integration spec to make sure the brand refresh is applied properly

Closes #411

Identifying a user need

Required for the GOV.UK brand refresh, happening across the GOV.UK estate on 25th June 2025.

Notes

There's a deeper discussion about what codebases using the tech docs gem but not the GOV.UK brand should look like, which we're not gonna solve with this PR. This brand updates project has identified a possible whiff around things that aren't GOV.UK-branded things co-opting bits of the GOV.UK brand eg: the black header and blue bar and where they fit into the GOV.UK ecosystem (should they even?). The Design System team are hoping to take a look at this eventually.

We should probably remove that extra test or incorperate it into the first test in the integration spec once govuk-frontend v6 is released and we remove the brand refresh feature flags.

@owenatgov owenatgov marked this pull request as ready for review June 16, 2025 09:56
Copy link
Member

@romaricpascal romaricpascal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking grand, only comment is a little nitpick to make the tests a little more succint 😊

@owenatgov owenatgov force-pushed the apply-brand-refresh branch from 8e962e8 to 1cfca9e Compare June 16, 2025 15:14
@owenatgov owenatgov merged commit 4f8e630 into main Jun 16, 2025
3 checks passed
@owenatgov owenatgov deleted the apply-brand-refresh branch June 16, 2025 15:16
@romaricpascal romaricpascal mentioned this pull request Jun 17, 2025
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

Successfully merging this pull request may close these issues.

Write production implementation of the brand refresh

3 participants