Skip to content
Commit b4199d9a authored by Aran Ink's avatar Aran Ink
Browse files

Only attach long-press handlers to LongPressActions.

Previously, all actions created views with a long press handler. Now, we only add long press handlers to the views if the action supports them, which prevents TalkBack from speaking long press instructions for actions that have no long press behavior.

This does not account for actions which have a long press handler which does not provide any user feedback, either because the result is not visible, or because it does nothing due to some other restriction, like user privileges. In the future, I think we should remove these long press behaviors where possible, since they're inscrutible to all users, and even moreso to users with visual impairments.

Test: Manual. Turn on talkback and verify that the Emergency action does not speak instructions to "double tap and hold to long press." Other actions with a long press handler (like Bug Report or Restart) should still speak these instructions.

Fixes: 147861227
Change-Id: I203bd0ac661a3912389ee584a863eb9f773b2927
parent b41e7e8f
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment