Skip to content

fix(a11y): enforce 44pt minimum tap targets on back buttons and touchables - #115

Merged
cybermax4200 merged 1 commit into
ecotask-network:mainfrom
Anichris-koded:fix/task-detail-touch-target
Aug 22, 2026
Merged

fix(a11y): enforce 44pt minimum tap targets on back buttons and touchables#115
cybermax4200 merged 1 commit into
ecotask-network:mainfrom
Anichris-koded:fix/task-detail-touch-target

Conversation

@Anichris-koded

Copy link
Copy Markdown
Contributor

Summary

Fixes the back button on TaskDetailScreen (and every other screen with one) having no minimum tap target. The buttons were plain text inside a TouchableOpacity with no padding, well below the 44pt minimum required by Apple's HIG / Android's Material Design.

Changes

  • Back buttons (TaskDetailScreen, SubmitProofScreen, SendTokensScreen, EditProfileScreen): consistent styling with paddingVertical: spacing.sm, paddingHorizontal: spacing.md, minHeight: 44, and accessibilityRole="button" (VoiceOver now announces "Back"/"Cancel, button").
  • Audit of other sub-44pt touchables: retry/try-again buttons (TaskDetail, TaskList, Map, TransactionHistory), filter/sort/radius chips (TaskList, Map), copy + expand buttons (PublicKeyDisplay), Retry in PendingProofsBanner, and "Grant Permission" in SubmitProofScreen now all meet the 44pt minimum.
  • Back text is visually aligned via a compensating negative left margin so the wider hit area doesn't shift the label.

Acceptance criteria

  • All back buttons have minimum 44pt tap targets
  • Consistent back button styling across all screens
  • All TouchableOpacity instances meet minimum tap target size
  • VoiceOver announces the button as "Back" with accessibilityRole="button"

Verification

  • npm run typecheck passes
  • npm test passes (40 suites, 335 tests)

Closes #97

…ables

Give every back button (TaskDetail, SubmitProof, SendTokens, EditProfile)
a consistent minimum 44pt tap target with padding and accessibilityRole,
and bump other sub-44pt touchables (retry buttons, filter chips, copy
button) to meet the same minimum.

Closes ecotask-network#97

@cybermax4200 cybermax4200 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work!

@cybermax4200
cybermax4200 merged commit f85b044 into ecotask-network:main Aug 22, 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.

[UX] TaskDetailScreen back button has no minimum tap target — fails accessibility guidelines

2 participants