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

Auditing fixes #3421

Merged
merged 2 commits into from
Oct 27, 2023
Merged

Auditing fixes #3421

merged 2 commits into from
Oct 27, 2023

Conversation

andysellick
Copy link
Contributor

⚠️ This repo is Continuously Deployed: make sure you follow the guidance ⚠️

What / why

Fixes some issues in this application relating to component use, as raised by our component auditing tools. See individual commits for details.

Visual changes

None.

- the Javascript for the intervention component was not being included in this application
- this line was present to override the appearance of list item elements within govspeak, rendered in the header of the /coronavirus page
- there are currently no list item elements in the header of the coronavirus page, and since the page is not now being frequently updated it seems safe to remove this style override
- we shouldn't be overriding component styles like this anyway, if there's a problem with the component it should be fixed in the component
@govuk-ci govuk-ci temporarily deployed to collections-pr-3421 October 23, 2023 07:38 Inactive
@andysellick andysellick merged commit ff32b02 into main Oct 27, 2023
@andysellick andysellick deleted the auditing-fixes branch October 27, 2023 09:48
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.

3 participants