Skip to content
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

Merged
merged 2 commits into from
May 13, 2021
Merged

Update link styles #1014

merged 2 commits into from
May 13, 2021

Conversation

vanitabarrett
Copy link
Contributor

@vanitabarrett vanitabarrett commented Apr 30, 2021

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.

@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-prototype-kit-pr-1014 May 4, 2021 13:54 Inactive
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-prototype-kit-pr-1014 May 5, 2021 12:29 Inactive
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-prototype-kit-pr-1014 May 6, 2021 07:58 Inactive
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-prototype-kit-pr-1014 May 6, 2021 12:29 Inactive
Copy link
Member

@lfdebrux lfdebrux left a 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.

@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-prototype-kit-pr-1014 May 7, 2021 09:14 Inactive
Copy link
Member

@hannalaakso hannalaakso left a 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 👍

@vanitabarrett
Copy link
Contributor Author

Note: need to delete the pre-release before merging

@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-prototype-kit-pr-1014 May 12, 2021 12:31 Inactive
@vanitabarrett
Copy link
Contributor Author

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

@vanitabarrett vanitabarrett marked this pull request as ready for review May 13, 2021 15:10
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-prototype-kit-pr-1014 May 13, 2021 15:10 Inactive
@vanitabarrett vanitabarrett merged commit cfcb4f9 into main May 13, 2021
@vanitabarrett vanitabarrett deleted the update-link-styles branch May 13, 2021 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement the new link and hover styles in the prototype kit
5 participants