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

Remove anchor styling in govuk-lists #985

Merged
merged 2 commits into from
Sep 6, 2018

Conversation

NickColley
Copy link
Contributor

@NickColley NickColley commented Sep 6, 2018

These styles partially implement the .govuk-link styles, but fail to implement the correct focus styles.

Additionally this need can be satisfied by enabling global styles, which is on by default in the GOV.UK Prototype Kit.

I have checked our documentation and we do not recommend our users do this.

With this both in mind this feels reasonable to remove this functionality as a breaking change, without a deprecation period.

This need can be satisfied by enabling global styles, which is on by default in the GOV.UK Prototype Kit.

I have checked our [documentation](https://design-system.service.gov.uk/styles/typography/#lists) and we do not recommend our users do this.

With this both in mind this feels reasonable to remove this functionality as a breaking change.
@NickColley NickColley added this to the v2.0.0 milestone Sep 6, 2018
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-985 September 6, 2018 15:40 Inactive
Copy link
Contributor

@36degrees 36degrees left a comment

Choose a reason for hiding this comment

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

👍 Merge once changelog'd.

@@ -103,6 +103,16 @@
```
([PR #981](https://github.com/alphagov/govuk-frontend/pull/981))

- Remove anchor styling in govuk-lists
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

@NickColley NickColley merged commit 83f8a79 into master Sep 6, 2018
@NickColley NickColley deleted the remove-global-govuk-list-anchor-styles branch September 6, 2018 15:50
@NickColley NickColley mentioned this pull request Sep 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants