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

Enable new link styles by default #3599

Merged
merged 2 commits into from
May 10, 2023
Merged

Conversation

36degrees
Copy link
Contributor

In July 2021, we made some improvements to new link styles which were released as part of v3.12.0.

We introduced these changes behind a feature flag $govuk-new-link-styles: true which service teams needed to set before importing GOV.UK Frontend in order to opt-in.

This was in case users:

Now that #2204 has been fixed in Chromium, we can make the new link styles ‘opt-out’.

Closes #2350

@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-3599 May 10, 2023 09:07 Inactive
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-3599 May 10, 2023 09:08 Inactive
In July 2021, we made some improvements to new link styles [1] which were released as part of v3.12.0.

We introduced these changes behind a feature flag `$govuk-new-link-styles: true` which service teams needed to set before importing GOV.UK Frontend in order to opt-in.

This was in case users:

- needed to make changes to their service to introduce these new link styles consistently
- wanted to avoid opting in if they have links within a multiple-column layout in your CSS, due to Known issue: hover state is mis-painted on links within CSS multiple-column layouts #2204

Now that #2204 has been fixed in Chromium, we can make the new link styles ‘opt-out’.

[1]: #2350
@36degrees 36degrees force-pushed the enable-new-link-styles-default branch from 5aff848 to 7647591 Compare May 10, 2023 09:10
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-3599 May 10, 2023 09:10 Inactive
Copy link
Member

@romaricpascal romaricpascal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good for me, just thinking it's also an opportunity to mark deprecation and forget about the setting until the next breaking change if that's what we want 😊

Comment on lines +13 to +23
#### New link styles are now enabled by default

In GOV.UK Frontend v3.12.0 we introduced new link styles which:

- have underlines that are consistently thinner and a bit further away from the link text
- have a clearer hover state, where the underline gets thicker to make the link stand out to users

The new link styles are now enabled by default. If you are setting `$govuk-new-link-styles` to `true` in your Sass you can now remove this.

This change was made in [pull request #3599: Enable new link styles by default](https://github.com/alphagov/govuk-frontend/pull/3599).

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

question Did we say this setting would be removed in the next breaking change? If that's the case, do we want to add a little note that the setting is deprecated?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would appreciate your thoughts on this in #3600! If we don't want to do #3600 then yes, we should probably deprecate it instead.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, I'll merge this and if we do decide to deprecate it we can do that in a follow up PR too.

@36degrees 36degrees merged commit 1e7b7e2 into main May 10, 2023
@36degrees 36degrees deleted the enable-new-link-styles-default branch May 10, 2023 09:33
@romaricpascal romaricpascal mentioned this pull request Dec 8, 2023
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.

Enable new link styles by default
3 participants