Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Accessibility: Screen reader focus stays on the open menu button when the note actions menu is opened #10253

Closed
personalizedrefrigerator opened this issue Apr 2, 2024 · 0 comments · Fixed by #11031
Closed

Accessibility: Screen reader focus stays on the open menu button when the note actions menu is opened #10253

personalizedrefrigerator opened this issue Apr 2, 2024 · 0 comments · Fixed by #11031
Labels
accessibility Related to accessibility bug It's a bug medium Medium priority issues mobile All mobile platforms

Comments

@personalizedrefrigerator
Copy link
Collaborator

personalizedrefrigerator commented Apr 2, 2024 •
edited
Loading

Operating system

Windows

Joplin version

3.0.2

Desktop version info

No response

Current behaviour

To reproduce:

  1. Enable TalkBack
  2. Open a note
  3. Focus the note actions button (top right of screen)
  4. Activate it (double-tap)
  5. Move focus to the next item (swipe right)
  6. Move focus to the previous item (swipe left)
  7. Observe that the announced items are the toggle button for opening/closing the menu and whatever comes after it.

Expected behaviour

Focus should jump to the note menu when opening it.

Related to instea/react-native-popup-menu#138, but it should still be possible to fix by moving accessibility focus ourselves.

Logs

No response
@personalizedrefrigerator personalizedrefrigerator added bug It's a bug mobile All mobile platforms accessibility Related to accessibility medium Medium priority issues labels Apr 2, 2024
Sidd-R added a commit to Sidd-R/joplin that referenced this issue Apr 19, 2024
@Sidd-R
fix laurent22#10253 : move focus to menu option
962bf79
@Sidd-R Sidd-R mentioned this issue Apr 19, 2024
Closed
This was referenced Jul 24, 2024
Merged
Open
personalizedrefrigerator added a commit to personalizedrefrigerator/joplin that referenced this issue Sep 11, 2024
@personalizedrefrigerator
Mobile: Fixes laurent22#10253: move accessibility focus to the first …
c678310 
…note action menu item on open
Merged
Merged
Open
70 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
accessibility Related to accessibility bug It's a bug medium Medium priority issues mobile All mobile platforms
Projects
None yet
Milestone
No milestone
1 participant
@personalizedrefrigerator