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

docs(examples): update example css to reflect real-life usage #1064

Merged
merged 2 commits into from
Jan 10, 2025

Conversation

chrispymm
Copy link
Contributor

The code in our examples was using application.css which is a sass compiled file including all of govuk frontend, moj-frontend and all the custom styles for the docs site. This could cause potential issues where our own css could affect components.

This PR refactors things such that the example layout includes compiled govuk-frontend and (separately) compiled moj-frontend and none of the docs site css.

This ensure we are presenting (and testing) components in a more isolated way with no danger of style leakage.

The code in our examples was using application.css, which is a sass compiled file including all of
govuk frontend, moj-frontend and all the custom styles for the docs site.  This could cause
potential issues where our own css could affect components. This commit refactors things such that
the example layout includes compiled govuk-frontend and (separately) compiled moj-frontend and none
of the docs site css. This ensure we are presenting (and testing) components in a more isolated way
with no danger of style leakage.
@chrispymm chrispymm requested a review from a team as a code owner January 9, 2025 14:49
@chrispymm chrispymm added the preview:request Deploy PR to a preview url label Jan 9, 2025
Copy link

github-actions bot commented Jan 9, 2025

🚀 Deployed to preview environment! If this is the first deploy, you may have to wait a few minutes for your preview site to be ready on the following URL:

https://moj-frontend-refactor-example-layout-css.apps.live.cloud-platform.service.justice.gov.uk

Username: preview, Password: moj

@github-actions github-actions bot added preview:active and removed preview:request Deploy PR to a preview url labels Jan 9, 2025
Copy link

github-actions bot commented Jan 9, 2025

🚀 Deployed to preview environment! If this is the first deploy, you may have to wait a few minutes for your preview site to be ready on the following URL:

https://moj-frontend-refactor-example-layout-css.apps.live.cloud-platform.service.justice.gov.uk

Username: preview, Password: moj

@chrispymm chrispymm merged commit dd472f9 into main Jan 10, 2025
19 checks passed
@chrispymm chrispymm deleted the refactor-example-layout-css branch January 10, 2025 09:57
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