-
Notifications
You must be signed in to change notification settings - Fork 236
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 link styles #1014
Update link styles #1014
Conversation
b25decf
to
c96e5d4
Compare
c96e5d4
to
844f572
Compare
844f572
to
8b1aaad
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.
This looks good to me, the new link styles seem to work as expected. I checked the main page and the step by step navigation examples on Chrome, Safari, and Firefox.
8b1aaad
to
9856a99
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.
@vanitabarrett Thanks for addressing my comments. This looks good to me 👍
Note: need to delete the pre-release before merging |
9856a99
to
0c60c21
Compare
Build will be failing now as the pre-release has been removed. Needs to be rebased once the new version of GOV.UK Frontend has been added |
0c60c21
to
b87d2fe
Compare
Closes #1012
NOTE: This PR currently points at a pre-release of GOV.UK Frontend. This commit should be removed and replaced by a separate PR to bump GOV.UK Frontend to v3.12.0 before this PR is merged.
What
Update the prototype kit to opt into the new link styles.
Update the step-by-step navigation template to make use of the new link styles.
Why
The new link and hover styles will be opt-in until the next major release of GOV.UK Frontend. However, we'd like the prototype kit to take advantage of the new styles when 3.12.0 is released in order to present the most up-to-date version of GOV.UK Frontend.