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 GOV.UK Frontend and use new link styles #232

Merged
merged 4 commits into from
May 17, 2021

Conversation

lfdebrux
Copy link
Member

@lfdebrux lfdebrux commented May 14, 2021

GOV.UK Frontend v3.12.0 comes with new (optional) link styles that are designed to be easier to read and to have a clearer hover state [1].

This PR updates the govuk-frontend npm package from 3.6.0 to 3.12.0 and opts-in to the new link styles for the tech-docs-gem stylesheet. We also refactor the table of contents navigation and search results links to use new the new link style mixins [2] so that they are styled consistently with other links.

Note that this commit makes a small design change; search result links are now underlined when not hovered.

Part of alphagov/govuk-design-system#1641

@lfdebrux lfdebrux changed the title Bump govuk-frontend from 3.6.0 to 3.12.0 Update GOV.UK Frontend and use new link styles May 14, 2021
lfdebrux added 2 commits May 17, 2021 09:03
Updated package.json and ran `npm update`, following the steps in the
README.

Bumps [govuk-frontend](https://github.com/alphagov/govuk-frontend/):
  - [Release notes](https://github.com/alphagov/govuk-frontend/blob/main/CHANGELOG.md)
  - [Commits](alphagov/govuk-frontend@v3.6.0...v3.12.0)
GOV.UK Frontend v3.12.0 comes with new (optional) link styles that are
designed to be easier to read and to have a clearer hover state [[1]].

This commit opts-in to the new link styles for the tech-docs-gem
stylesheet, and refactors the table of contents navigation and search
results links to use new the new link style mixins [[2]] so that they
are styled consistently with other links.

Note that this commit makes a small design change; search result links
are now underlined when not hovered.

[[1]]: https://github.com/alphagov/govuk-frontend/blob/main/CHANGELOG.md#links-are-easier-to-read-and-have-a-clearer-hover-state-opt-in
[[2]]: https://frontend.design-system.service.gov.uk/sass-api-reference/#helpers-links
@lfdebrux lfdebrux force-pushed the ldeb-update-govuk-frontend branch from 60b2f64 to 1a8f4c5 Compare May 17, 2021 08:03
@lfdebrux lfdebrux force-pushed the ldeb-update-govuk-frontend branch from 1a8f4c5 to 9c2f55a Compare May 17, 2021 08:17
lfdebrux added 2 commits May 17, 2021 09:18
govuk-frontend 3.7.0 changes the way that Sass dependencies work, so
that CSS compiles faster [[1]].

This commit changes the Sass file for the tech docs gem to take
advantage of the new system, by importing the component `index`
stylesheets which don't include dependencies multiple times.

This commit also changes the imports of govuk-frontend to import
`govuk/base` instead of importing the settings, tools, and helpers by
hand, as this will make it easier to migrate this repo to GOV.UK
Frontend v4.0.0 when it is released.

[1]: https://github.com/alphagov/govuk-frontend/blob/main/CHANGELOG.md#sass-now-compiles-faster
@lfdebrux lfdebrux requested a review from vanitabarrett May 17, 2021 11:51
@lfdebrux lfdebrux merged commit 1f8e761 into master May 17, 2021
@lfdebrux lfdebrux deleted the ldeb-update-govuk-frontend branch May 17, 2021 15:17
@lfdebrux lfdebrux mentioned this pull request May 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

3 participants