Skip to content

chore: add accessibility to DailySetup views#3

Merged
minghinmatthewlam merged 1 commit intomainfrom
chore/accessibility-dailysetup
Mar 9, 2026
Merged

chore: add accessibility to DailySetup views#3
minghinmatthewlam merged 1 commit intomainfrom
chore/accessibility-dailysetup

Conversation

@minghinmatthewlam
Copy link
Copy Markdown
Owner

Summary

  • Added .accessibilityLabel("Close") and .accessibilityHint("Dismisses the daily setup without saving") to the xmark close button in DailySetupView
  • Added .accessibilityHint("Saves your focuses and starts the day") to the "Begin" button (label auto-derived from text)
  • FocusInputRow and DailySetupViewModel required no changes (TextField placeholder serves as label; ViewModel has no UI)

Test plan

  • Build succeeds (build_sim passed)
  • All 9 unit tests pass (test_sim passed)
  • VoiceOver walkthrough: verify close button reads "Close — Dismisses the daily setup without saving"
  • VoiceOver walkthrough: verify Begin button reads "Begin — Saves your focuses and starts the day"

🤖 Generated with Claude Code

…lements

Add accessibilityLabel/accessibilityHint to the close button (xmark)
and accessibilityHint to the Begin button in DailySetupView.
@minghinmatthewlam minghinmatthewlam merged commit bf29a10 into main Mar 9, 2026
4 checks passed
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