shahondin1624/ShadowrunCharSheet
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 1 Wiki Activity

[2.3] Accessible Color Contrast on Cards #7

New Issue
Closed
opened 2026-03-13 11:25:26 +01:00 by shahondin1624 · 0 comments
shahondin1624 commented 2026-03-13 11:25:26 +01:00
Owner
Copy Link

User Story

As a user, I want text on color-coded cards to be legible in both themes.

Acceptance Criteria

  • Replace if (isInDarkMode) Color.Black else Color.White with luminance-based contrast calculation
  • Minimum WCAG 4.5:1 contrast ratio
  • Utility function contrastTextColor(backgroundColor) in theme package

Notes

Affects Attribute.kt and Talent.kt card text colors. Current approach may fail for mid-luminance colors.

## User Story **As a** user, **I want** text on color-coded cards to be legible in both themes. ## Acceptance Criteria - [ ] Replace `if (isInDarkMode) Color.Black else Color.White` with luminance-based contrast calculation - [ ] Minimum WCAG 4.5:1 contrast ratio - [ ] Utility function `contrastTextColor(backgroundColor)` in theme package ## Notes Affects `Attribute.kt` and `Talent.kt` card text colors. Current approach may fail for mid-luminance colors.
shahondin1624 added this to the Phase 7: Polish milestone 2026-03-13 11:25:26 +01:00
shahondin1624 added the epic:cards-reworkpriority:should-have labels 2026-03-13 11:28:03 +01:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
epic:cards-rework
Epic: Attribute & Talent cards rework
 epic:character-display
Epic: Character info display
 epic:character-management
Epic: Character management (editing, persistence)
 epic:dice-rolling
Epic: Dice rolling UX
 epic:navigation
Epic: Navigation & page structure
 epic:responsive-layout
Epic: Responsive layout foundation
 epic:testing
Epic: Automated UI testing
 priority:must-have
Priority: Must-have for MVP
 priority:nice-to-have
Priority: Nice-to-have, polish and extras
 priority:should-have
Priority: Should-have, important but not blocking
No Label epic:cards-rework priority:should-have
Milestone
No items
No Milestone Phase 7: Polish
Projects
Clear projects
No project
Assignees
Clear assignees
shahondin1624
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: shahondin1624/ShadowrunCharSheet#7
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?