Skip to content

[Project Solar / Phase 1 / Components] TooltipButton carbonization#3622

Open
dchyun wants to merge 8 commits intoproject-solar/phase-1-main-feature-branchfrom
project-solar/phase-1/dchyun/tooltip-carbonization
Open

[Project Solar / Phase 1 / Components] TooltipButton carbonization#3622
dchyun wants to merge 8 commits intoproject-solar/phase-1-main-feature-branchfrom
project-solar/phase-1/dchyun/tooltip-carbonization

Conversation

@dchyun
Copy link
Contributor

@dchyun dchyun commented Feb 17, 2026

📌 Summary

This PR carbonizes the TooltipButton component and the hdsTooltip modifier and adds component tokens for them to the tokens library.

Tooltip carbonization page

🛠️ Detailed description

Summary of changes

  • Aligned typography, colors, paddings, other foundational styles
  • Align caret icons (see notes below)

One significant change is to the caret icon that is rendered next to the tooltip. Currently the tippjs config for the tooltip adds an svg for the caret icon. In Carbon however, the caret is rendered with css along using clip-path: polygon(0 100%, 50% 0, 100% 100%);.

To accommodate both of these use cases the SVG for the tooltip icon in HDS has been converted to be rendered through a clip-path using the path() value. A variable --token-tooltip-caret-clip-path has been created with a clip-path value for each theme.

🔗 External links

Jira ticket: HDS-5062
Figma file: [if it applies]


👀 Component checklist

  • Percy was checked for any visual regression

💬 Please consider using conventional comments when reviewing this PR.

📋 PCI review checklist
  • If applicable, I've documented a plan to revert these changes if they require more than reverting the pull request.
  • If applicable, I've worked with GRC to document the impact of any changes to security controls.
    Examples of changes to controls include access controls, encryption, logging, etc.
  • If applicable, I've worked with GRC to ensure compliance due to a significant change to the in-scope PCI environment.
    Examples include changes to operating systems, ports, protocols, services, cryptography-related components, PII processing code, etc.

@vercel
Copy link

vercel bot commented Feb 17, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
hds-showcase Ready Ready Preview Feb 25, 2026 3:18pm
hds-website Ready Ready Preview Feb 25, 2026 3:18pm

Request Review

@didoo
Copy link
Contributor

didoo commented Feb 17, 2026

@dchyun consider rebasing this on #3572 so you have all the latest foundational tokens

}
}
},
"color": {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

[Note] There are existing tooltip variables of the format color.foreground.primary and color.surface.primary. This does not match the expected format for color variables. Additional color variables foreground.color and surface.color have been added and leveraged in the component.

For now I have left these existing color variables included, as I believe it would be a breaking change to remove them, but they could be removed in the future.

@dchyun dchyun force-pushed the project-solar/phase-1/dchyun/tooltip-carbonization branch from e6d762c to 19e80ca Compare February 24, 2026 15:46
@dchyun dchyun force-pushed the project-solar/phase-1/dchyun/tooltip-carbonization branch from 0b33772 to fe5b5da Compare February 24, 2026 20:20
@dchyun dchyun force-pushed the project-solar/phase-1/dchyun/tooltip-carbonization branch from 7401572 to 4dbc9d2 Compare February 24, 2026 21:25
@dchyun dchyun force-pushed the project-solar/phase-1/dchyun/tooltip-carbonization branch from 4dbc9d2 to de5f2e4 Compare February 24, 2026 21:31
@dchyun dchyun force-pushed the project-solar/phase-1/dchyun/tooltip-carbonization branch from de5f2e4 to 8816d75 Compare February 24, 2026 21:46
@dchyun dchyun force-pushed the project-solar/phase-1/dchyun/tooltip-carbonization branch from 8816d75 to b6866f0 Compare February 25, 2026 14:30
@dchyun dchyun force-pushed the project-solar/phase-1/dchyun/tooltip-carbonization branch from b6866f0 to 4992d47 Compare February 25, 2026 14:57
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.

4 participants