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

Suppress Sass warnings for $legacy deprecated colour palette #2358

Merged
merged 2 commits into from
Oct 11, 2023

Conversation

colinrotherham
Copy link
Contributor

@colinrotherham colinrotherham commented Oct 9, 2023

This PR suppresses Sass $legacy colour palette deprecation warnings logged by GOV.UK Frontend v5 until the following files and packages can be updated from issue #2293:

  • govuk-prototype-kit/lib/assets/sass/patterns/_pagination.scss
  • govuk-prototype-kit-step-by-step/sass/_step-by-step-navigation.scss
  • govuk-prototype-kit-step-by-step/sass/_step-by-step-navigation-header.scss

This completes GOV.UK Frontend v5 plugin support in #2293

Legacy colour palette

The legacy palette was replaced in GOV.UK Frontend v3.0.0:
https://designnotes.blog.gov.uk/2019/07/29/weve-updated-the-gov-uk-colours-and-font/

Support has continued throughout v4.0.0 but is deprecated in v5.0.0

@colinrotherham colinrotherham changed the base branch from main to nunjucks-env October 9, 2023 13:32
@colinrotherham colinrotherham changed the title Suppress GOV.UK Frontend $legacy deprecation warnings Suppress Sass warnings for $legacy deprecated colour palette Oct 9, 2023
BenSurgisonGDS
BenSurgisonGDS previously approved these changes Oct 9, 2023
We have various package files using `$legacy` parameters now deprecated in GOV.UK Frontend v5

We should suppress these deprecation warnings until the following files can be updated:

* `govuk-prototype-kit/lib/assets/sass/patterns/_pagination.scss`
* `govuk-prototype-kit-step-by-step/sass/_step-by-step-navigation.scss`
* `govuk-prototype-kit-step-by-step/sass/_step-by-step-navigation-header.scss`
@colinrotherham
Copy link
Contributor Author

Moved the CHANGELOG entry into Unreleased

nataliecarey
nataliecarey previously approved these changes Oct 10, 2023
Base automatically changed from nunjucks-env to main October 11, 2023 09:14
@colinrotherham colinrotherham dismissed stale reviews from nataliecarey and BenSurgisonGDS October 11, 2023 09:14

The base branch was changed.

@colinrotherham colinrotherham merged commit 67acce8 into main Oct 11, 2023
27 checks passed
@colinrotherham colinrotherham deleted the frontend-v5-deprecated branch October 11, 2023 09:18
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.

3 participants