New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update the header #557
Update the header #557
Conversation
443882d
to
700ad7e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Left two comments regarding the tag 🏷
<% if product_name %> | ||
<span class="govuk-header__product-name"> | ||
<%= product_name %> | ||
</span> | ||
<% end %> | ||
</a> | ||
|
||
<% unless environment == "production" %> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we move this inside the anchor so Assistive Tech users know what the environment is when they get to the landmark?
app/assets/stylesheets/govuk_publishing_components/components/_layout-header.scss
Show resolved
Hide resolved
app/assets/stylesheets/govuk_publishing_components/components/_layout-header.scss
Show resolved
Hide resolved
This will not need to be configurable because all the apps that use this will be part of the "GOV.UK Publishing" product.
There's a bit of a consensus that we want to keep the red for production. Personally I super disagree - I think the production version should be a proud GOV.UK blue.
This adds the environment indicator (staging, integration, development) to the header. From user research we've seen that it might prevent confusion, especially because we currently show the tag in all interfaces.
700ad7e
to
454a4a0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
Aw I like the blue too. Could we maintain the customisability of the name and whether the environment label is there in this heading, but default to "Publishing"? Reason I ask is that we've been trying to use this header for the component guide itself. |
@kevindew let's try to keep the header consistent across the apps, including component guide. If it does look weird we can re-instate the blue. |
This is to allow Signon to be called "Signon" again. We'll still use "Publishing" as the default. Follow up to #557. https://trello.com/c/05wMNlxg
This is to allow Signon to be called "Signon" again. We'll still use "Publishing" as the default. Follow up to #557. https://trello.com/c/05wMNlxg
This:
product_name
is no longer configurable)https://trello.com/c/05wMNlxg