Skip to content

[UI]: Migrate react-tooltip usages to CustomTooltip from @sistent/sistent#7788

Open
Maanvi212006 wants to merge 6 commits into
layer5io:masterfrom
Maanvi212006:fix/migrate-react-tooltip-to-custom-tooltip
Open

[UI]: Migrate react-tooltip usages to CustomTooltip from @sistent/sistent#7788
Maanvi212006 wants to merge 6 commits into
layer5io:masterfrom
Maanvi212006:fix/migrate-react-tooltip-to-custom-tooltip

Conversation

@Maanvi212006

Copy link
Copy Markdown
Contributor

Description

This PR fixes #7751

Replaced all usages of react-tooltip with CustomTooltip from @sistent/sistent across the codebase, consistent with the project's design system and existing usage in src/components/CopyValue/index.js.

Files changed:

  • src/components/blog-view-tooltip.js
  • src/components/Landscape-Table/index.js
  • src/components/SMI-Table/index.js
  • src/sections/Landscape/categories.js
  • src/sections/Learn-Layer5/Chapters/index.js

Notes for Reviewers
All 5 files pass syntax check and no react-tooltip imports remain in the codebase

Signed commits

  • Yes, I signed my commits.

Signed-off-by: Maanvi Chetwani <maanvichetwani21@gmail.com>
@github-actions github-actions Bot added the area/landscape The service mesh landscape label Jun 10, 2026

@rishiraj38 rishiraj38 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The PR preview is failing due to build errors.
Please check the build logs and try fixing those errors.

@Maanvi212006

Copy link
Copy Markdown
Contributor Author

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:

  • Can't resolve './LandscapeTable.style'
  • Can't resolve '../../assets/images/landscape/passing.svg'
  • Can't resolve '../../assets/images/landscape/failing.svg'

These imports were already present in src/components/Landscape-Table/index.js before my changes. This PR only replaces react-tooltip imports with CustomTooltip from @sistent/sistent — no paths or imports were added or modified beyond that.

Could you confirm whether these errors exist on the master branch as well? Happy to investigate further if needed!

rishiraj38 and others added 4 commits June 16, 2026 19:25
Signed-off-by: Maanvi Chetwani <maanvichetwani21@gmail.com>
Signed-off-by: Maanvi Chetwani <maanvichetwani21@gmail.com>
Signed-off-by: Maanvi Chetwani <maanvichetwani21@gmail.com>
@github-actions

github-actions Bot commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

🚀 Preview deployment: https://layer5.io/pr-preview/pr-7788/

Note: Preview may take a moment (GitHub Pages deployment in progress). Please wait and refresh. Track deployment here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/landscape The service mesh landscape

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[UI]: Migrate react-tooltip usages to CustomTooltip from @sistent/sistent

2 participants