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 govuk frontend to 3.4.0 #65

Merged
merged 16 commits into from
Dec 23, 2019
Merged

Update govuk frontend to 3.4.0 #65

merged 16 commits into from
Dec 23, 2019

Conversation

frankieroberto
Copy link
Contributor

@frankieroberto frankieroberto commented Nov 11, 2019

This updates the govuk-frontend library from version 2.13.0 to 3.3.0. This mainly means that the colours have changed.

The page weight of the compiled CSS has also been reduced.

Review app: https://psd-pr-65-web.london.cloudapps.digital/

@edwardhorsford
Copy link
Contributor

Many thanks for doing this work @frankieroberto!

I've had a good look through the app and there are a few snags - some likely because it highlights gaps in our codebase.

  • Missing underline on skip link in header.

  • Footer text now using two colours - should all be dark I think.

  • Force underlines on footer links (override design system).

  • Focussed hover for white buttons isn’t nice (could consider a separate issue as our existing hover for them isn’t good).

  • Focus State slightly broken on autocomplete inputs - no thick black border. I suspect if we updated our autocompletes this might fix this issue. They're not using full design system styles.

  • Hover state of delete buttons - should get darker (design system has introduced delete buttons since we made ours).

  • Need better focus state for autocomplete clear button (suggest thick yellow background, black border bottom)

  • Keycloak styles will also need updating (different app though)

@frankieroberto
Copy link
Contributor Author

@edwardhorsford so the "Missing underline on skip link in header" is a bug within the govuk-frontend code itself, specifically when "global styles" are enabled.

Has been reported to the Design System team here: alphagov/govuk-frontend#1641

@edwardhorsford
Copy link
Contributor

edwardhorsford commented Dec 20, 2019

  • Missing underline on skip link in header.
    Bug with design system - reported to them. In the short term I suggest leaving as is

  • Footer text now using two colours - should all be dark I think.

  • Force underlines on footer links (override design system).
    Design system team have an update coming that resolves this.

  • Hover state of delete buttons - should get darker (design system has introduced delete buttons since we made ours).
    Swapped to using govuk-button--warning

  • Need better focus state for autocomplete clear button (suggest thick yellow background, black border bottom)

Will raise as separate bug

  • Focussed hover for white buttons isn’t nice (could consider a separate issue as our existing hover for them isn’t good).

Will fix in separate PR

  • Focus State slightly broken on autocomplete inputs - no thick black border. I suspect if we updated our autocompletes this might fix this issue. They're not using full design system styles.

Will raise as separate bug

  • Keycloak styles will also need updating (different app though)

Will raise as separate story Raised

Copy link
Contributor

@edwardhorsford edwardhorsford left a comment

Choose a reason for hiding this comment

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

Looks great! many thanks @frankieroberto

@frankieroberto frankieroberto changed the title Update govuk frontend to 3.3.0 Update govuk frontend to 3.4.0 Dec 20, 2019
@slorek slorek merged commit ca56ccf into master Dec 23, 2019
@slorek slorek deleted the update-govuk-frontend branch December 23, 2019 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants