Skip to content
This repository has been archived by the owner on Aug 16, 2024. It is now read-only.

feat(branding): update crown logo to Tudor Crown logo in header #55

Merged
merged 2 commits into from
Feb 14, 2024

Conversation

sanakhandvsa
Copy link
Contributor

@sanakhandvsa sanakhandvsa commented Feb 8, 2024

Description

This PR updates the branding in our olcs-selfserve header by replacing the existing crown logo with the new Tudor Crown logo.
logo changed based on instruction given on here govuk-frontend

Changes

  • Updated the SVG path for the logo in the header component.
  • Changed the source file name in the corresponding .phtml file to reflect the new GOV.UK logo.

Because we are not using the Nunjucks macro, so we have to locate the HTML for the existing crown and replace it with provided updated HTML.

Related issue: https://dvsa.atlassian.net/browse/VOL-4960

Before submitting (or marking as "ready for review")

  • Does the pull request title follow the conventional commit specification?
  • Have you performed a self-review of the code
  • Have you have added tests that prove the fix or feature is effective and working
  • Did you make sure to update any documentation relating to this change?

@sanakhandvsa sanakhandvsa self-assigned this Feb 8, 2024
@JoshuaLicense JoshuaLicense merged commit 45e5ca9 into main Feb 14, 2024
3 of 6 checks passed
@JoshuaLicense JoshuaLicense deleted the vol-4960-update-govuk-logo branch February 14, 2024 15:57
jerotire pushed a commit to dvsa/vol-app that referenced this pull request Aug 16, 2024
* feat(branding): update crown logo to Tudor Crown logo in header

* fixup! feat(branding): update crown logo to Tudor Crown logo in header

---------

Co-authored-by: JoshuaLicense <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants