Skip to content

Conversation

@krishnavbajoria02
Copy link
Contributor

Notes for Reviewers

This PR fixes #1269

Description:
This PR refactors and standardizes the InfoCircle icon implementation in Sistent to align with existing icon patterns.

Changes Made

  1. Refactored index.tsx → index.ts
    • index.tsx contained the full InfoIcon component implementation.
    • Replaced with a minimal index.ts .
  2. Updated InfoCircleIcon.tsx
    • Replaced default WHITE fill to KEPPEL_GREEN_FILL.
    • Import source updated.

Screenshot/Image:
Screenshot (520)

Signed commits

  • Yes, I signed my commits.

krishnavbajoria02 and others added 2 commits February 7, 2026 09:55
Signed-off-by: Krishnav Bajoria <127018567+krishnavbajoria02@users.noreply.github.com>
@krishnavbajoria02
Copy link
Contributor Author

The build failure appears to be related to MUI/Emotion SSR on /extension/[...component] and not introduced by this icon change.

@Bhumikagarggg
Copy link

@krishnavbajoria02 Thank you for your contribution! Let's discuss this during the website call tomorrow at 6:30 PM IST | 7 AM CST Add it as an agenda item to the meeting minutes, if you would 🙂

@Rajesh-Nagarajan-11
Copy link
Member

@krishnavbajoria02 , Why workflow is failing ? -> Test with meshery Ui

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Add InfoIcon to Icon Collection

3 participants