[UI]: Migrate react-tooltip usages to CustomTooltip from @sistent/sistent#7788
[UI]: Migrate react-tooltip usages to CustomTooltip from @sistent/sistent#7788Maanvi212006 wants to merge 10 commits into
Conversation
Signed-off-by: Maanvi Chetwani <maanvichetwani21@gmail.com>
rishiraj38
left a comment
There was a problem hiding this comment.
The PR preview is failing due to build errors.
Please check the build logs and try fixing those errors.
|
Hi @rishiraj38, thanks for the review! I looked into the build errors and they appear to be pre-existing issues unrelated to this PR's changes. The failing errors are:
These imports were already present in Could you confirm whether these errors exist on the |
Signed-off-by: Maanvi Chetwani <maanvichetwani21@gmail.com>
Signed-off-by: Maanvi Chetwani <maanvichetwani21@gmail.com>
Signed-off-by: Maanvi Chetwani <maanvichetwani21@gmail.com>
|
Preview deployment for PR #7788 removed. This PR preview was automatically pruned because we keep only the 6 most recently updated previews on GitHub Pages to stay within deployment size limits. If needed, push a new commit to this PR to generate a fresh preview. |
Signed-off-by: Maanvi Chetwani <maanvichetwani21@gmail.com>
|
@rishiraj38 @chaitanyamedidar @Sbragul26 a gentle ping! |
Signed-off-by: Maanvi Chetwani <maanvichetwani21@gmail.com>
@rishiraj38 done |
Description
This PR fixes #7751
Replaced all usages of
react-tooltipwithCustomTooltipfrom@sistent/sistentacross the codebase, consistent with the project's design system and existing usage insrc/components/CopyValue/index.js.Files changed:
src/components/blog-view-tooltip.jssrc/components/Landscape-Table/index.jssrc/components/SMI-Table/index.jssrc/sections/Landscape/categories.jssrc/sections/Learn-Layer5/Chapters/index.jsNotes for Reviewers
All 5 files pass syntax check and no
react-tooltipimports remain in the codebaseSigned commits