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 deprecated govuk-tag--inactive class #2417

Merged
merged 2 commits into from
Nov 8, 2021

Conversation

lfdebrux
Copy link
Member

@lfdebrux lfdebrux commented Nov 3, 2021

Fixes #1757 (also see here for details of how this is a breaking change for users).

Class govuk-tag--inactive was replaced by govuk-tag--grey and deprecated in PR #1711.

This commit removes the class completely. Users should replace any uses of this class with .govuk-tag--grey.

@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-2417 November 3, 2021 13:51 Inactive
@lfdebrux lfdebrux force-pushed the ldeb-remove-govuk-tag--inactive branch from 6dfcb32 to ddeee24 Compare November 3, 2021 13:52
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-2417 November 3, 2021 13:52 Inactive
@lfdebrux
Copy link
Member Author

lfdebrux commented Nov 3, 2021

I've had a go at writing the changelog message, @EoinShaughnessy you might want to take a look as well?

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.

💥 🗑️ 🎉

CHANGELOG.md Outdated Show resolved Hide resolved
@@ -60,6 +60,6 @@ examples:
data:
text: This is a new service – your feedback will help us to improve it
tag:
classes: govuk-tag--inactive
Copy link
Contributor

Choose a reason for hiding this comment

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

Good catch 👍🏻

@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-2417 November 4, 2021 10:16 Inactive
@lfdebrux lfdebrux force-pushed the ldeb-remove-govuk-tag--inactive branch from ed2d8e1 to 3149232 Compare November 4, 2021 10:18
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-2417 November 4, 2021 10:18 Inactive
lfdebrux and others added 2 commits November 4, 2021 10:18
Class `govuk-tag--inactive` was replaced by `govuk-tag--grey` and
deprecated in PR #1711. This commit replaces references to it in our
tests, examples, and fixtures.
Class govuk-tag--inactive was replaced by govuk-tag--grey and deprecated
in PR #1711.

This commit removes the class completely. Users should replace any uses
of this class with `.govuk-tag--grey`.

Co-authored-by: Oliver Byford <oliver.byford@digital.cabinet-office.gov.uk>
@lfdebrux lfdebrux force-pushed the ldeb-remove-govuk-tag--inactive branch from 3149232 to 5d2fa1e Compare November 4, 2021 10:19
@lfdebrux lfdebrux changed the title Remove deprecated .govuk-tag--inactive class Remove deprecated govuk-tag--inactive class Nov 4, 2021
@lfdebrux lfdebrux changed the title Remove deprecated govuk-tag--inactive class Remove deprecated govuk-tag--inactive class Nov 4, 2021
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-2417 November 4, 2021 10:19 Inactive
@EoinShaughnessy
Copy link
Contributor

@lfdebrux Changelog update looks great!

@lfdebrux lfdebrux merged commit 0c4485b into main Nov 8, 2021
@lfdebrux lfdebrux deleted the ldeb-remove-govuk-tag--inactive branch November 8, 2021 09:41
@vanitabarrett vanitabarrett mentioned this pull request Dec 15, 2021
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.

Remove govuk-tag--inactive class
4 participants