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

Override nhsuk- brand prefix on notification banner and panel components #1672

Merged
merged 1 commit into from
Sep 18, 2024

Conversation

paulrobertlloyd
Copy link
Contributor

@paulrobertlloyd paulrobertlloyd commented Aug 23, 2024

Trialing the change to govuk-components proposed at x-govuk/govuk-components#552 (see x-govuk/govuk-components#547)

This allows us to use govuk- brand prefix for components that do not have associated styles in nhsuk-frontend, and thus not need to extend the govuk- prefixed classes to apply to components that have been forced to use nhsuk- prefix given the global brand setting for this gem.

There may be a corresponding update the GOV.UK Form Builder gem; that will allow us to review and remove styles for the password input and file upload form fields in a similar fashion.

Will merge once these upstream updates have been published. /cc @peteryates

@tvararu tvararu temporarily deployed to mavis-pr-1672 August 23, 2024 15:35 Inactive
@tvararu tvararu force-pushed the govuk-components-brand branch from 65c7bf9 to 8bd01a0 Compare September 18, 2024 14:03
@tvararu tvararu marked this pull request as ready for review September 18, 2024 14:04
@tvararu tvararu requested a review from a team as a code owner September 18, 2024 14:04
@tvararu tvararu enabled auto-merge (rebase) September 18, 2024 14:04
@tvararu
Copy link
Member

tvararu commented Sep 18, 2024

Thanks @paulrobertlloyd!

Copy link

@tvararu tvararu merged commit c35c6f4 into main Sep 18, 2024
11 checks passed
@tvararu tvararu deleted the govuk-components-brand branch September 18, 2024 14:12
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