-
Notifications
You must be signed in to change notification settings - Fork 25
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 page template examples #1975
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: Frankie Roberto <[email protected]>
Co-authored-by: Frankie Roberto <[email protected]>
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.
Hi @anandamaryon1, I've made a suggestion for the text re the larger h1
on content pages.
I couldn't work out how to suggest a change to the wording under Transactional pages that says: "We also recommended you use a smaller page heading size using the .nhsuk-heading-l
modifier class."
Please could we change it to "We also recommend you use a smaller h1
heading size using the .nhsuk-heading-l
modifier class."
Co-authored-by: Sara Wilcox <[email protected]>
@sarawilcox I've made your suggested change. |
Release at same time as: #1974 |
Description
Related to #1974, this makes a few updates to the page template examples:
nhsuk-heading-l
and put it inside a 2/3 column grid.nhsuk-heading-xl
to the content page exampleFixes #1969
Related issue
Checklist
package.json
(using SEMVER)