Skip to content

WIP: Refactor content style guide#1741

Closed
Chris Pymm (chrispymm) wants to merge 5 commits intomainfrom
simplify-content-style-guide
Closed

WIP: Refactor content style guide#1741
Chris Pymm (chrispymm) wants to merge 5 commits intomainfrom
simplify-content-style-guide

Conversation

@chrispymm
Copy link
Contributor

Builds on the work in simplify content style guide to refactor the style guide page itself.

Each style guide entry becomes its own .md file

---
title: entry title
---
Entry content markdown

We then use an 11tydata file to assign a 'tag' to all the entries to group them into an eleventy collection, and also use eleventyComputed to generate a a data entry for the first letter of the title

The style guide layout file can then wrangle the data from the collection into a format that can be passed into the govuk-accordion nunjucks macro.

This hugely simplifies the addition and editing of entries in the style guide, as we will now only be dealing directly with the content in .md files, not a big mess of HTML.

@chrispymm Chris Pymm (chrispymm) requested a review from a team as a code owner September 3, 2025 21:30
@chrispymm Chris Pymm (chrispymm) added the preview:request Deploy PR to a preview url label Sep 3, 2025
@github-actions
Copy link

github-actions bot commented Sep 3, 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-pr-1741.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 Sep 3, 2025
@chrispymm Chris Pymm (chrispymm) force-pushed the simplify-content-style-guide-updates-page branch from 048b36c to 2ed548a Compare September 5, 2025 05:27
Base automatically changed from simplify-content-style-guide-updates-page to main September 16, 2025 10:47
@chrispymm Chris Pymm (chrispymm) linked an issue Sep 26, 2025 that may be closed by this pull request
2 tasks
@chrispymm
Copy link
Contributor Author

Closed and replaced with #1847

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant