Merged
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
Adds new “circle outline” add/remove icons to the toolkit icon set, wiring them into the icon manifest, generated sprite, and the docs-site icon gallery data so they can be consumed by components and displayed in the design system site.
Changes:
- Added two new SVG assets: add/remove circle outline variants.
- Registered the new icons in
packages/toolkit/assets/icons/manifest.jsonandicon-sprite.svg. - Updated the docs-site icon ordering list to include the new icons.
Reviewed changes
Copilot reviewed 2 out of 5 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/toolkit/assets/icons/AddCircleOutlineOutlined.svg | Adds new add-circle outline SVG asset. |
| packages/toolkit/assets/icons/RemoveCircleOutlineOutlined.svg | Adds new remove-circle outline SVG asset. |
| packages/toolkit/assets/icons/manifest.json | Registers the two new icons in the toolkit icon manifest. |
| packages/toolkit/assets/icons/icon-sprite.svg | Adds sprite <symbol> entries for the two new icons. |
| packages/site/views/_data/materialIcons.js | Includes the new icons in the docs gallery ordering. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Bump the toolkit package for the circle outline icon addition. Refresh the changelog, upgrading guide, and release versioning docs for the standalone patch release.
Clarify that review units are independently releasable by default. Require release-impact assessment, versioning, changelog work, and release-contract checks before sign-off.
Rename the workflow concept to merge readiness in the prompt copy and make the PR title/body guidance intentional instead of mechanically section-driven.
rmds16
approved these changes
Apr 9, 2026
Hard-rename the toolkit and React icon surface to the current Figma names, including the sprite assets, docs gallery, and component/example usages.
Document the breaking icon rename release across toolkit and react-components, update package versions, and refresh migration guidance.
Sort docs gallery names alphabetically within each icon category and keep Storybook icon dropdowns alphabetical for easier review.
Keep the Stepper gallery rows in 0 through 9 order while leaving the other icon categories alphabetical.
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 DS-372 icon naming sync across the toolkit icon asset set, the docs-site icon gallery, the public React
Iconsurface, release metadata, and the merge-readiness prompt follow-up.After comparing the current repo against the Figma icon page, the scope expanded beyond the two circle-outline additions because the public icon names were no longer aligned with the design-system source of truth. This branch takes the hard-cut route while there are still no downstream consumers to preserve: the old names are removed rather than kept as compatibility aliases.
It also carries the merge-readiness prompt follow-up that came out of this work, so future review units default to independently releasable branches and more intentional PR metadata.
Ticket:
Release scope
@ourfuturehealth/toolkit->4.9.0(planned tag:toolkit-v4.9.0)@ourfuturehealth/react-components->0.7.0(planned tag:react-v0.7.0)site/docschanges are included in-repo and are not independently versionedBreaking Changes
Done->CheckExpandLess->ChevronUpExpandMore->ChevronDownCalendarTodayOutlined->CalendarOutlineFmdGoodOutlined->LocationOutlineLinkedIn->LinkedinUPGRADING.mdin this branch for the full rename table and migration examples.Key Changes
1) Public icon names are now Figma-aligned end to end
AddCircleOutlineandMinusCircleOutlineLooksZerothroughLooksNine) with the Figma-alignedListZerothroughListNine2) Current docs and component consumers now use the renamed icon surface
Iconstories/tests and current React component consumers to use the renamed names3) Release and workflow follow-up are included in the same branch
@ourfuturehealth/toolkitto4.9.0and@ourfuturehealth/react-componentsto0.7.0CHANGELOG.md,UPGRADING.md, and release-versioning updates for the breaking rename releaseValidation
Automated:
npm testpnpm lintpnpm buildpnpm docs:release-contractpnpm smoke:release-artifactsManual QA:
16,24, and32, includingCheck,ChevronUp,ChevronDown,ClockOutline,LocationOutline,AddCircleOutline, andMinusCircleOutlineIconandCardStorybook surfaces use the renamed options correctlyReviewer Focus
manifest.json, generatedicon-sprite.svg, docs gallery, and current toolkit/React consumers4.9.0 / 0.7.0breaking-release metadata and migration guidance