-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
docs(tooltip): revert to peviously correct stories, use help icon #18245
docs(tooltip): revert to peviously correct stories, use help icon #18245
Conversation
✅ Deploy Preview for v11-carbon-web-components ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for carbon-elements ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for v11-carbon-react ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #18245 +/- ##
=======================================
Coverage 84.16% 84.16%
=======================================
Files 404 404
Lines 14350 14350
Branches 4649 4649
=======================================
Hits 12077 12077
Misses 2112 2112
Partials 161 161 ☔ View full report in Codecov by Sentry. |
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.
Yes, I think this is better and more accurately reflects how we talk about the default (standard) tooltip in our docs. Thanks for helping with this Taylor.
181f38f
Follow up to #17006, this reverts the tooltip stories back to correct examples that use the help icon instead of the information icon (which previously created confusion)
Related:
Tooltip
&DefinitionTooltip
changed storybook and functionality #15887Changelog
Changed
Testing / Reviewing