-
Notifications
You must be signed in to change notification settings - Fork 0
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
CCMSPUI-411: Upgrade GOVUK Frontend to 5.7.1 #304
Conversation
Signed-off-by: Jamie Briggs <jamie.briggs@digital.justice.gov.uk>
Signed-off-by: Jamie Briggs <jamie.briggs@digital.justice.gov.uk>
…y/ccmspui-411-govuk-5.7.1 # Conflicts: # src/main/resources/scss/ccms.scss
… service navigation component. Fixed references to fragment to be <th:block/> elements to coincide with this change. Signed-off-by: Jamie Briggs <jamie.briggs@digital.justice.gov.uk>
…nto story/ccmspui-411-govuk-5.7.1
Signed-off-by: Jamie Briggs <jamie.briggs@digital.justice.gov.uk>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
I've just mereged my changes which has a new caab.js file. Are you able to merge the contents of that file into the all.js? |
@PhilDigitalJustice Not currently. Would need to implement However you did highlight that gulpfile.js is copying |
… the correct location Signed-off-by: Jamie Briggs <jamie.briggs@digital.justice.gov.uk>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Upgrading GOVUK Frontend from
5.3.1
to5.7.1
.Changes
5.5.0
: GOVUK Frontend includes updated colour paleteNot much change to frontend for this inclusion, rather using the colour palette directly in
.scss
where a colour is used.5.6.0
: Updated header to use the services component5.7.0
: Crest updated to use King Charles III logoThis change was partially done when CCMSPUI-408 was merged, however part of this PR it's been checked properly.
Before:
After:
Changes to GDS but not to project
Listing changes to GDS which did not effect the project but were worth noting as part of my findings.
5.4.1
: Breadcrumbs to usenav
instead ofdiv
and to add an aria-label which was introduced in.5.5.0
: There is a class now added which helps wrap longer words which would usually trail off screen. You can do this by wrapping the content with the classgovuk-!-text-break-word
.