commit a9eae7e6b840f2e48e05cd06a3a03567decdb31e
author calderwoodra <calderwoodra@google.com> Fri Sep 15 17:00:28 2017 -0700
committer Eric Erfanian <erfanian@google.com> Mon Sep 18 07:54:01 2017 -0700
tree 72ac9512da41f6ee50078a1e7905f9ad27009b1b
parent cceef72738753b896d9cc8e99d05b04a612a5fee

Dialpad VM and + are now only accessible w/ double-tap instead of long hover.

Users using touch to explore with TalkBack on the dialpad would occasionally
call voicemail if they hovered over 1 for a long period of time. This
functionality doesn't align with keyboards, where a long hover still acts as a
normal click. This CL updates the Dialpad to align better with a keyboard and
now users can only speeddial voicemail and type a 0 with double-tap activation.

Bug: 62712616
Test: manual
PiperOrigin-RevId: 168909308
Change-Id: Ib46342f173aef7c2ef64c2b42745966f83266b98

java/com/android/dialer/dialpadview/DialpadKeyButton.java

diff --git a/java/com/android/dialer/dialpadview/DialpadKeyButton.java b/java/com/android/dialer/dialpadview/DialpadKeyButton.java
index 24ca9cc..0c842eb 100644
--- a/java/com/android/dialer/dialpadview/DialpadKeyButton.java
+++ b/java/com/android/dialer/dialpadview/DialpadKeyButton.java
@@ -167,17 +167,15 @@
           break;
         case MotionEvent.ACTION_HOVER_EXIT:
           if (mHoverBounds.contains(event.getX(), event.getY())) {
-            if (mLongHovered) {
-              performLongClick();
-            } else {
-              simulateClickForAccessibility();
-            }
+            simulateClickForAccessibility();
           }
 
           cancelLongHover();
           setClickable(mWasClickable);
           setLongClickable(mWasLongClickable);
           break;
+        default: // No-op
+          break;
       }
     }