-
Notifications
You must be signed in to change notification settings - Fork 2k
feat(tooltip): react|wc parity #21018
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
feat(tooltip): react|wc parity #21018
Conversation
✅ Deploy Preview for v11-carbon-web-components ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for carbon-elements ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for v11-carbon-react ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify project configuration. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #21018 +/- ##
==========================================
- Coverage 92.59% 92.58% -0.01%
==========================================
Files 515 515
Lines 38219 38219
Branches 5864 5802 -62
==========================================
- Hits 35387 35385 -2
- Misses 2682 2685 +3
+ Partials 150 149 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
✅ Deploy Preview for v11-carbon-web-components ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for carbon-elements ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for v11-carbon-react ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Hey @maradwan26 looking great, just noticed some things: WC:
Screen.Recording.2025-11-14.at.17.25.55.mov |
Fixed ✅
Thanks for noticing! This is an issue for
I fixed it here✅ and double checked to ensure there isn't any regression to the other components. |
|
Also noticed the |
heloiselui
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just noticed a few minor things, but LGTM! 🚀
packages/web-components/src/components/tooltip/tooltip.stories.ts
Outdated
Show resolved
Hide resolved
packages/web-components/src/components/tooltip/tooltip-story.scss
Outdated
Show resolved
Hide resolved
packages/web-components/src/components/tooltip/tooltip-story.scss
Outdated
Show resolved
Hide resolved
Co-authored-by: Heloise Lui <[email protected]>
Co-authored-by: Heloise Lui <[email protected]>
Co-authored-by: Heloise Lui <[email protected]>
Closes #20838
Closes #20963
Fixes bugs for
cds-tooltip, aligns WC stories with React, and adds controls to all storiesChangelog
React
New
TODOcomments to remove thehighContrastprop in the next major releasehighContrastshould always betruefor the Tooltip component, it should not be configurableChanged
alignprop to use-start-endalignments instead of the deprecated-left-top-right-bottomRemoved
Web Components
New
autoaligncds-popovercaret during bottom and right alignments when usingborderandautoalign, I fixed it in this PR, too.top/bottom-start/endalignments where the caret and content would get misaligned when thecds-tooltip-contenttext was shortChanged
Testing / Reviewing
Web Components
React
PR Checklist
As the author of this PR, before marking ready for review, confirm you:
[ ] Wrote passing tests that cover this change