-
Notifications
You must be signed in to change notification settings - Fork 17
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
PLATUI-2705: Upgrade to latest govuk-frontend v5.0.0 #338
Conversation
35d4ea5
to
52574ce
Compare
PLATUI-2627: Trying to fix JS import PLATUI-2627: Changed init of govuk-frontend JS
…ges in govuk-frontend
52574ce
to
fd4d686
Compare
fd4d686
to
b3fe3ac
Compare
tasks/gulp/compile-js.js
Outdated
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.
probably lots of improvements we could snag from https://github.com/alphagov/govuk-frontend/blob/60b0a67bcd7cbb9c072c0fd573c30cdd5c2d31ba/packages/govuk-frontend/rollup.publish.config.mjs I think it's outside scope of this change probably - though maybe some bits we might want to do
b3fe3ac
to
bced47f
Compare
@@ -247,15 +247,6 @@ describe('header', () => { | |||
|
|||
describe('SVG logo', () => { | |||
const $ = render('header', {}); | |||
const $svg = $('.govuk-header__logotype-crown'); | |||
|
|||
it('sets focusable="false" so that IE does not treat it as an interactive element', () => { |
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.
This test has been deleted but it looks like we still have the SVG with focusable="false", should we keep the test? Or delete the attribute?
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 replaced the SVG element with the one from the guidance on the govuk-frontend release notes. I am going to leave the test that tests for focusable="false" as this is what govuk recommends.
bced47f
to
6e55d21
Compare
Stuff that has happened in this PR