Skip to content

[NO TICKET] Add Long-term Care Icon#3964

Merged
jack-ryan-nava-pbc merged 8 commits intoCMSgov:mainfrom
kamiTT:add-long-term-care-icon
Apr 20, 2026
Merged

[NO TICKET] Add Long-term Care Icon#3964
jack-ryan-nava-pbc merged 8 commits intoCMSgov:mainfrom
kamiTT:add-long-term-care-icon

Conversation

@kamiTT
Copy link
Copy Markdown
Contributor

@kamiTT kamiTT commented Mar 6, 2026

Summary

Adding a new icon that CXT uses within their application: longterm care

How to test

  1. Build and serve storybook.
  2. Navigate to the medicare icons component
  3. Confirm new icon appears: LongtermCareIcon

Checklist

  • Prefixed the PR title with the Jira ticket number as [CMSDS-####] Title or [NO-TICKET] if this is unticketed work.
  • Selected appropriate Type (only one) label for this PR, if it is a breaking change, label should only be Type: Breaking
  • Selected appropriate Impacts, multiple can be selected.
  • Selected appropriate release milestone

If this is a change to design:

  • If visual regression image references have been changed, design MUST be assigned to review. In this instance, designer approval is a requirement before the PR can be merged.

If this is a change to code:

  • Created or updated unit tests to cover any new or modified code
  • Verified that running both npm run test:unit and npm run test:browser:all were each successful
  • If necessary, updated unit-test snapshots (npm run test:unit:update) and browser-test snapshots (npm run test:browser:all:update)

If this is a change to documentation/content:

  • Checked for spelling and grammatical errors
  • Communicate the assigned milestone/release date with Design so they can communicate appropriately

@kamiTT kamiTT changed the title add longterm care icon [NO TICKET] Add Long-term Care Icon Mar 10, 2026
@jack-ryan-nava-pbc
Copy link
Copy Markdown
Collaborator

not a wolf

@jack-ryan-nava-pbc jack-ryan-nava-pbc added Impacts: Medicare Impacts the Medicare theme directly. Type: Added Indicates a new feature. labels Mar 26, 2026
@jack-ryan-nava-pbc jack-ryan-nava-pbc added this to the 14.0.0 milestone Mar 26, 2026
@jack-ryan-nava-pbc jack-ryan-nava-pbc self-assigned this Apr 10, 2026
@jack-ryan-nava-pbc jack-ryan-nava-pbc marked this pull request as ready for review April 10, 2026 15:45
export { default as LongtermCareIcon } from './LongtermCareIcon';
=======
export { default as NursingHomeIcon } from './NursingHomeIcon';
>>>>>>> main
Copy link
Copy Markdown
Collaborator

@tamara-corbalt tamara-corbalt Apr 10, 2026

Choose a reason for hiding this comment

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

Looks like git markers made into this file as well.


const defaultProps = {
className: '',
title: 'Long-term Care',
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Just a quick reminder to use translate functionality on the title prop.

defaultTitle: 'Nursing Home',
component: <NursingHomeIcon />,
name: 'NursingHomeIcon',
>>>>>>> main
Copy link
Copy Markdown
Collaborator

@tamara-corbalt tamara-corbalt Apr 10, 2026

Choose a reason for hiding this comment

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

Looks like unresolved Git conflict markers made it into this file (<<<<<<<, =======, >>>>>>>).

@kamiTT kamiTT force-pushed the add-long-term-care-icon branch from 735db6a to 83680a9 Compare April 10, 2026 19:34
@kamiTT kamiTT requested a review from tamara-corbalt April 10, 2026 19:34
Copy link
Copy Markdown
Collaborator

@tamara-corbalt tamara-corbalt left a comment

Choose a reason for hiding this comment

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

Looks good! Thanks for resolving the merge conflicts and updating the snapshots.

The only remaining change is to use the translate utility for the title prop, and then we should be good to go.

@kamiTT kamiTT requested a review from tamara-corbalt April 20, 2026 14:20
@jack-ryan-nava-pbc jack-ryan-nava-pbc merged commit fc5d977 into CMSgov:main Apr 20, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Impacts: Medicare Impacts the Medicare theme directly. Type: Added Indicates a new feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants