Skip to content

fix(handicap): ensure the screen reader reads the popup content in the correct order#1237

Merged
danji90 merged 11 commits intomasterfrom
daniel/fixAmoScreenReader
Nov 12, 2025
Merged

fix(handicap): ensure the screen reader reads the popup content in the correct order#1237
danji90 merged 11 commits intomasterfrom
daniel/fixAmoScreenReader

Conversation

@danji90
Copy link
Copy Markdown
Contributor

@danji90 danji90 commented Nov 11, 2025

How to

Others

  • It's not a hack or at least an unauthorized hack :).
  • The images added are optimized.
  • Everything in ticket description has been fixed.
  • The author of the MR has made its own review before assigning the reviewer.
  • The title means something for a human being and follows the conventional commits specification.
  • The title contains [WIP] if it's necessary.
  • Labels applied. if it's a release? a hotfix?
  • Check the preview app on iOS before merge: Is everything working? SVGs visible?
  • Tests added.

@danji90 danji90 self-assigned this Nov 11, 2025
@danji90 danji90 added the bug Something isn't working label Nov 11, 2025
@vercel
Copy link
Copy Markdown

vercel Bot commented Nov 11, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
dev-trafimage-maps Ready Ready Preview Comment Nov 12, 2025 11:40am
trafimage-maps Ready Ready Preview Comment Nov 12, 2025 11:40am

@oterral
Copy link
Copy Markdown
Collaborator

oterral commented Nov 12, 2025

lgtm I have tested with the ubuntu screen reader and it seems better as before.

@danji90 danji90 merged commit d5b6839 into master Nov 12, 2025
6 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Development

Successfully merging this pull request may close these issues.

2 participants