Open
Conversation
✅ Deploy Preview for ofh-design-system-storybook ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for ofh-design-system-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Pull request overview
Updates the design-system Summary List to match the latest FE treatment, adds explicit spacing/border options to the toolkit macro, and introduces a first public React SummaryList component with matching variants.
Changes:
- Refreshed toolkit
summary-listmarkup + styles, addingpadded/noBordermacro options and action-link attribute support. - Added React
SummaryListcomponent (exports, stories, unit + a11y tests) aligned to toolkit structure and modifiers. - Updated design-system docs, examples, versioning docs, upgrading guide, and changelog for the new release versions and component guidance.
Reviewed changes
Copilot reviewed 19 out of 19 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/toolkit/package.json | Bumps toolkit version to 4.12.0 for the release. |
| packages/toolkit/components/summary-list/template.njk | Adds padded/compact + no-border logic, row classes, and action-link attributes support. |
| packages/toolkit/components/summary-list/_summary-list.scss | Reworks summary list layout to grid-based rows and updates spacing/action styling. |
| packages/toolkit/components/summary-list/README.md | Updates summary-list docs to teach compact/no-border + new macro options. |
| packages/site/views/examples/components/summary-list/without-border.njk | Updates example to use compact spacing (padded: false). |
| packages/site/views/design-system/components/summary-list/without-border/index.njk | Updates design-system example to use compact spacing (padded: false). |
| packages/site/views/design-system/components/summary-list/macro-options.json | Documents new padded/noBorder and action attributes options. |
| packages/site/views/design-system/components/summary-list/index.njk | Refreshes component guidance text and update date. |
| packages/react-components/src/index.ts | Exports the new SummaryList component and its types. |
| packages/react-components/src/components/SummaryList/index.ts | Adds component-level barrel exports for SummaryList. |
| packages/react-components/src/components/SummaryList/SummaryList.tsx | Introduces the React SummaryList implementation with padded/compact/no-border variants. |
| packages/react-components/src/components/SummaryList/SummaryList.test.tsx | Adds unit tests and an axe accessibility check for the new component. |
| packages/react-components/src/components/SummaryList/SummaryList.stories.tsx | Adds Storybook stories covering default/compact/without-actions/without-border. |
| packages/react-components/package.json | Bumps react-components version to 0.10.0 for the release. |
| packages/react-components/README.md | Updates install URL and adds SummaryList to usage + component list. |
| docs/release-versioning-strategy.md | Updates example tags/URLs and the release history table. |
| docs/consuming-react-components.md | Adds SummaryList to the documented exported components list. |
| UPGRADING.md | Adds a v4.12.0 / React v0.10.0 upgrade section and migration notes. |
| CHANGELOG.md | Adds changelog entries for toolkit 4.12.0 and react-components 0.10.0. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
jpsanders101
approved these changes
Apr 15, 2026
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
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.
Description
This PR delivers DSE-331 summary list refresh across toolkit, React, the docs site, and Storybook.
It aligns the summary list with the current Figma treatment, introduces the first public React
SummaryListcomponent, and updates Storybook so the summary list stories follow the newerDocs/Default/Builder/ showcase teaching pattern.Ticket: DSE-331
Release scope
@ourfuturehealth/toolkit->4.12.0@ourfuturehealth/react-components->0.11.0CHANGELOG.md,UPGRADING.md,docs/release-versioning-strategy.md,docs/consuming-react-components.md, andpackages/react-components/README.mdBreaking Changes
Key Changes
summary-listcomponent to the current Figma structure across the default, compact, without-actions, and no-border treatmentspadded: falseandnoBorder: truewhile preserving compatibility for existingofh-summary-list--no-borderusageSummaryListcomponent with optional action links, compact spacing, and no-border supportrowsshape and the distinction between real props and the Storybook-onlyshowActionshelperDefaultas a realistic fixed example,Builderas the interactive surface, and the other stories as fixed showcase examplesValidation
npm testpnpm lintpnpm buildpnpm docs:release-contractpnpm smoke:release-artifactspnpm --filter=@ourfuturehealth/react-components build:storybookReviewer Focus
paddedandnoBordermacro options versus existing class-based compatibilitySummaryListAPI and therowsshape guidance should be easy for consumers to followtoolkit-v4.12.0/react-v0.11.0