🏗 refactor(svg): consolidate raw hex colors with chartColors.ts constants#14043
Conversation
Consolidate 53 raw hex color literals across 5 mission-control SVG components to use named imports from lib/theme/chartColors.ts. - Add slate, provider brand, and accent color constants to chartColors.ts - Update BlueprintDefs, ClusterZone, DependencyPath, PhaseTimeline, ProjectNode - Zero visual changes — exact same hex values via named constants Fixes #14041 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Signed-off-by: kubestellar-architect[bot] <kubestellar-architect[bot]@users.noreply.github.com>
kubestellar-hive
Bot
added
architect
kubestellar-prow
Bot
added
the
dco-signoff: yes
✅ Deploy Preview for kubestellarconsole ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
👋 Hey @kubestellar-hive[bot] — thanks for opening this PR!
This is an automated message. |
kubestellar-prow
Bot
added
the
size/L
✅ Test Coverage CheckAll new source files in this PR have corresponding test files. Checked |
|
Thank you for your contribution! Your PR has been merged. Check out what's new:
Stay connected: Slack #kubestellar-dev | Multi-Cluster Survey |
❌ Post-Merge Verification: failedCommit: |
|
Post-merge build verification passed ✅ Both Go and frontend builds compiled successfully against merge commit |
Summary
Consolidate 53 raw hex color literals across 5 mission-control SVG components to use named imports from
lib/theme/chartColors.ts.Changes
web/src/lib/theme/chartColors.ts— Add 22 new named constants: slate palette (200-950), provider brand colors (EKS, GKE, AKS, etc.), accent colors (indigo-200/400, purple-300, orange-200, emerald-300/900, green-400, sky-500), plus WHITE/BLACKBlueprintDefs.tsx— Replace 8 raw hex → named constantsClusterZone.tsx— Replace 21 raw hex → named constants (COLORS + PROVIDER_COLORS objects)DependencyPath.tsx— Replace 8 raw hex → named constantsPhaseTimeline.tsx— Replace 9 raw hex → named constantsProjectNode.tsx— Replace 10 raw hex → named constantsZero visual changes
Every named constant maps to the exact same hex value. This is a pure mechanical extraction for maintainability and theme consistency.
Fixes #14041