-
Notifications
You must be signed in to change notification settings - Fork 20
Align inverse headers with the new styles of the gem component #5034
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
base: main
Are you sure you want to change the base?
Conversation
b7644e7 to
84cdef7
Compare
c279f33 to
4303e94
Compare
4303e94 to
40c4726
Compare
40c4726 to
54f4c2f
Compare
6913d6a to
c916ab6
Compare
c916ab6 to
1f1671d
Compare
1f1671d to
df6e0cc
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.
Nice work. All looks good to me!
| <% end %> | ||
| <%= render "govuk_publishing_components/components/lead_paragraph", { | ||
| text: lead_para, | ||
| inverse: true, |
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.
Nice, I guess we should be able to fully remove this option once it's removed from https://github.com/alphagov/collections/blob/main/app/views/taxons/_page_header.html.erb#L17.
As the new style inverse header (with light blue background) doesn't require white text, remove setting it with inverse styles. Add class to links to style with the new inverse header style black.
A design decision was made to retain the blue background on the CSV previews for now, reflecting the fact that the context is different than for the full page header that will be light blue with the new style inverse header.
df6e0cc to
b0fbdac
Compare
What
Remove the white text styles from the Service Manual and Service Toolkit headers, as well as making links black.
Update the CSV preview to use the HTML publication variant of the inverse header.
Why
This aligns the typography styles inside the header on the Service Manual and Service Toolkit with the updated the new style gem inverse header component.
A design decision was made to retain the deeper blue header colour in the context of document views for now, so the header on the CSV preview will use the HTML publication variant of the inverse header which is keeping its blue background.
https://gov-uk.atlassian.net/jira/software/c/projects/NAV/boards/1422?selectedIssue=NAV-18210
Screenshots?
The 'After' include the updates to to the gem component - see 'Depends on'.
Before
After
Before
After
Before
After (no difference)
Depends on
alphagov/govuk_publishing_components#5053 - I've added a Do Not Merge label until the gem PR has been merged.
Fixes https://gov-uk.atlassian.net/jira/software/c/projects/NAV/boards/1422?selectedIssue=NAV-18210.