Update template statistics "dashboard" tables to summary lists#6002
Open
kr8n3r wants to merge 7 commits into
Open
Update template statistics "dashboard" tables to summary lists#6002kr8n3r wants to merge 7 commits into
kr8n3r wants to merge 7 commits into
Conversation
kr8n3r
force-pushed
the
update-dashboard-tables-to-summary-lists
branch
2 times, most recently
from
July 22, 2026 09:08
a0b805d to
cacc38b
Compare
It's only used on template statistics page and as they're being converted into summary-lists, it can become a standalone macro without the need for table cell call. Sass also moved into a standalone partial
Used on template statistics page, but as we've migrate those tables to summary-lists, this className and styles are no longer coupled to a table, so it makes sense to move it out of `table.scss`.
Summary lists with table stats require some extra styles. - 50% width of the key element - template name link (or text for Precompiled PDFs) - template metadata tweaks Alternaive approach to this would be to use existing Design System override classes.
summary-list - Use govukSummaryList and custom modifier classes to apply styling - Replace use of table imports - Reimport standalone spark-bar component
summary-list - Use govukSummaryList and custom modifier classes to apply styling - Replace use of table imports - Reimport standalone spark-bar component
summary-list - Use govukSummaryList and custom modifier classes to apply styling - Replace use of table imports - Reimport standalone spark-bar component Used on /template-usage page
Account for summary-list HTML markup changes
kr8n3r
force-pushed
the
update-dashboard-tables-to-summary-lists
branch
from
July 22, 2026 10:59
cacc38b to
49957c2
Compare
kr8n3r
changed the title
kr8n3r
marked this pull request as ready for review
July 22, 2026 13:31
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Replace template stats "dashboard" tables with a summary list component. These stats are key:value pairs so from an accessibility standpoint, a summary list that's built on description list element is more appropriate.
Key value pairs each take 50% of the available space and we no longer truncate the template name.
A new summary list modifier class introduced to contain needed styling.
Styles and jinja macro moved out of
tableas they're now n longer used in the table context.Review
Visuals
Dashboard page: lazy load and stats
Template usage page
