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

Add sass via npm to compile stylesheets #19

Merged
merged 1 commit into from
Jan 25, 2024
Merged

Add sass via npm to compile stylesheets #19

merged 1 commit into from
Jan 25, 2024

Conversation

MatMoore
Copy link
Contributor

@MatMoore MatMoore commented Jan 24, 2024

This allows us to pass variables to the design system, and will make updating easier.

Use this to apply the design system styles globally, so that parts of the search result page can be rendered from markdown. If we want to restrict this more later, we can write more specific selectors that extend
the GOV.UK styles, like this:
https://github.com/ministryofjustice/cjs_scorecard_exploratory_analysis/blob/develop/cjs_test_app/scss/markdown-styles.scss

I've set up the package.json so we manually run npm run sass when we want to recompile the sass. I figure this is probably good enough for us as we won't need to change stylesheets very often.

This allows us to pass variables to the design system, and will make
updating easier.

Use this to apply the design system styles globally, so that parts of
the search result page can be rendered from markdown. If we want to
restrict this more later, we can write more specific selectors that extend
the GOV.UK styles, like this:
https://github.com/ministryofjustice/cjs_scorecard_exploratory_analysis/blob/develop/cjs_test_app/scss/markdown-styles.scss

I've set up the package.json so we manually run `npm run sass` when we
want to recompile the sass. I figure this is probably good enough for us
as we won't need to change stylesheets very often.
Copy link
Collaborator

@mitchdawson1982 mitchdawson1982 left a comment

Choose a reason for hiding this comment

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

Looks good...

@MatMoore MatMoore merged commit b599e5e into main Jan 25, 2024
2 checks passed
@MatMoore MatMoore deleted the add-sass branch January 25, 2024 09:40
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