Skip to content

refactor: extract all inline colors to WornColors#6

Merged
jvsena42 merged 1 commit into
mainfrom
refactor/extract-colors
Mar 24, 2026
Merged

refactor: extract all inline colors to WornColors#6
jvsena42 merged 1 commit into
mainfrom
refactor/extract-colors

Conversation

@jvsena42
Copy link
Copy Markdown
Owner

Summary

Centralize all inline color definitions into the WornColors theme files on both Android and iOS. This eliminates scattered hex color literals across 13 files and ensures a single source of truth for the design system palette.

Changes

  • Added 13 new color tokens to WornColors (Android) and WornColors.swift (iOS): AccentIndigo, AccentCoral, AccentGreenEnd, AccentGreenDark, DeleteRed, SaveGradientStart, SaveGradientEnd, and 6 CategoryDot* colors
  • Replaced all inline Color(0xFF...) / Color(hex: "...") usages across 13 component and screen files with WornColors.* references
  • Removed duplicate DeleteRed and AccentGreenEnd private vals from WardrobeScreen and OutfitsScreen
  • Made Category.iconRes() in AddItemFields internal (was already done in prior branch)
  • Clothing color picker palette (Cream, Black, Navy, etc.) intentionally left inline as those are data values, not theme colors

Test plan

  • Verify all screens render with correct colors (no visual regressions)
  • Check delete buttons are red, CTA gradients are green, outfit badges show indigo/coral/green
  • Verify category dots show correct colors per category in clothing cards
  • Run ./gradlew :shared:allTests — all tests pass
  • Run ./gradlew detekt — clean

Checklist

  • ./gradlew detekt passes
  • Tested on Android
  • Tested on iOS
  • Updated documentation (if applicable)

🤖 Generated with Claude Code

@jvsena42 jvsena42 enabled auto-merge March 24, 2026 00:11
@jvsena42 jvsena42 merged commit 7314b30 into main Mar 24, 2026
1 check passed
@jvsena42 jvsena42 deleted the refactor/extract-colors branch March 24, 2026 00:14
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.

1 participant