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

deps: Update govuk-frontend to version 4.5.0 #161

Merged
merged 1 commit into from
Feb 10, 2023
Merged

Conversation

jpveooys
Copy link
Contributor

@jpveooys jpveooys commented Feb 9, 2023

This updates govuk-frontend to version 4.5.0. The new version includes summary cards.

Unfortunately Chromium and Firefox have slightly different handling of hidden="until-found" (now used in the accordion), which required some tweaks to tests. (Chromium considers those elements as part of the accessibility tree, while other browsers don't as they haven't implemented support for hidden="until-found".)

@jpveooys jpveooys force-pushed the deps/govuk-frontend branch 2 times, most recently from e9a45ef to 95250cd Compare February 9, 2023 16:56
This updates govuk-frontend to version 4.5.0. The new version includes summary cards.

Unfortunately Chromium and Firefox have slightly different handling of `hidden="until-found"` (now used in the accordion), which required some tweaks to tests. (Chromium considers those elements as part of the accessibility tree, while other browsers don't seem to.)
@jpveooys jpveooys marked this pull request as ready for review February 10, 2023 09:06
@@ -4,6 +4,10 @@ import { expect, test } from '../fixtures'
// There are various known failures in the GOV.UK Design System:
// https://github.com/alphagov/govuk-frontend/issues/1280#issuecomment-509588851
const DISABLED_RULES: Record<string, Record<string, string[]>> = {
accordion: {
default: ['aria-allowed-attr'],
'with-summary-lines': ['aria-allowed-attr'],
Copy link
Contributor

Choose a reason for hiding this comment

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

I would add a comment because the issue is not included in the known issues link alphagov/govuk-frontend#1280 (comment)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It does have its own issue it seems alphagov/govuk-frontend#2472

@adamledwards adamledwards self-requested a review February 10, 2023 13:09
@jpveooys jpveooys merged commit d9002f8 into main Feb 10, 2023
@jpveooys jpveooys deleted the deps/govuk-frontend branch February 10, 2023 13:11
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.

2 participants