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

Extract published dates button group shared styles #2745

Merged
merged 1 commit into from
Apr 19, 2023

Conversation

jon-kirwan
Copy link
Contributor

@jon-kirwan jon-kirwan commented Apr 18, 2023

What

https://trello.com/c/fc41oyEK/1923-enable-individual-loading-of-stylesheets-in-government-frontend

Extract published dates button group shared styles

Why

These styles are currently included in _detailed-guide.scss but are used by 3 document types:

  • Consultation
  • Detailed guide
  • Publication

This will cause a problem when the AssetHelper is implemented since the complete _detailed-guide.scss styles would need to be loaded for the above document types.

Visual changes

None

Anything else

@govuk-ci govuk-ci temporarily deployed to government-frontend-pr-2745 April 18, 2023 10:12 Inactive
@jon-kirwan jon-kirwan force-pushed the extract-shared-styles branch from 057f062 to 9016fac Compare April 18, 2023 10:23
@govuk-ci govuk-ci temporarily deployed to government-frontend-pr-2745 April 18, 2023 10:23 Inactive
@jon-kirwan jon-kirwan requested a review from MartinJJones April 18, 2023 10:25
@jon-kirwan jon-kirwan marked this pull request as ready for review April 18, 2023 10:25
Copy link
Contributor

@MartinJJones MartinJJones left a comment

Choose a reason for hiding this comment

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

Nice find, the changes look good to me 👍

@jon-kirwan jon-kirwan force-pushed the extract-shared-styles branch from 9016fac to 4e0edbc Compare April 19, 2023 08:26
@govuk-ci govuk-ci temporarily deployed to government-frontend-pr-2745 April 19, 2023 08:26 Inactive
@jon-kirwan jon-kirwan merged commit 9b3395b into main Apr 19, 2023
@jon-kirwan jon-kirwan deleted the extract-shared-styles branch April 19, 2023 08:52
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