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

Remove @import of empty print stylesheets #2189

Merged
merged 1 commit into from
Jul 26, 2021

Conversation

danacotoran
Copy link
Contributor

There was some refactoring to remove display: none rules from component stylesheets in favour of using the govuk-!-display-none-print class instead: alphagov/govuk_publishing_components#1561

As a consequence of this, some of the component print stylesheets which previously used to contain CSS are now empty files. This removes references to these files from the application print stylesheet.

Relevant issue: alphagov/govuk_publishing_components#2065


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

@danacotoran danacotoran changed the title Remove @import of empty stylesheets Remove @import of empty print stylesheets Jul 23, 2021
@bevanloon bevanloon temporarily deployed to government-f-remove-emp-dnppoj July 23, 2021 10:56 Inactive
There was some refactoring to remove display: none rules from component
stylesheets in favour of using the govuk-!-display-none-print class
instead. alphagov/govuk_publishing_components#1561

As a consequence of this, some of the component print stylesheets which
previously used to contain CSS are now empty files. This removes
references to these files from the application print stylesheet.
@danacotoran danacotoran force-pushed the remove-empty-stylesheet-import branch from 4d6f4d0 to 1b746e9 Compare July 23, 2021 11:07
@bevanloon bevanloon temporarily deployed to government-f-remove-emp-dnppoj July 23, 2021 11:07 Inactive
@danacotoran danacotoran merged commit 8cd2f0c into main Jul 26, 2021
@danacotoran danacotoran deleted the remove-empty-stylesheet-import branch July 26, 2021 09:24
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