Skip to content

feat: improve text visibility over background images#50

Merged
kYem merged 2 commits into
mainfrom
feat/improve-visibility
Feb 1, 2026
Merged

feat: improve text visibility over background images#50
kYem merged 2 commits into
mainfrom
feat/improve-visibility

Conversation

@kYem
Copy link
Copy Markdown
Owner

@kYem kYem commented Jan 23, 2026

Summary

  • Add text shadows to quote text and author for better contrast against background images
  • Add variant prop to GoalInput component ('minimal' for direct background, 'boxed' for panels)
  • Use minimal variant in GoalFocusView (glass theme) with centered text and underline style
  • Keep boxed variant as default for panel/modal contexts (GoalsSection, GoalsList, GoalDetailView)

Test plan

  • Verify quote text is readable on various background images (light and dark areas)
  • Test GoalInput in focus mode (glass theme) - should show minimal underline style
  • Test GoalInput in full/compact mode (panel) - should show boxed style with Add button
  • Test adding tasks in both variants works correctly

- Add text shadows to quote text and author for better contrast
- Add variant prop to GoalInput ('minimal' for background, 'boxed' for panels)
- Redesign minimal variant with centered text and underline style
- Use minimal variant in GoalFocusView (glass theme direct background)
- Keep boxed variant as default for panel/modal contexts
@kYem kYem force-pushed the feat/improve-visibility branch from 236bd04 to 6cb7bed Compare January 23, 2026 07:17
Extract variant-specific styles into INDICATOR_STYLES and LINK_BUTTON_STYLES
constants. Create reusable GoalLinkButton and GoalPicker sub-components to
eliminate code duplication between minimal and boxed variants.
@kYem kYem merged commit 2ec98f7 into main Feb 1, 2026
1 check passed
@kYem kYem deleted the feat/improve-visibility branch February 1, 2026 15:10
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