From a9eae7e6b840f2e48e05cd06a3a03567decdb31e Mon Sep 17 00:00:00 2001
From: calderwoodra <calderwoodra@google.com>
Date: Fri, 15 Sep 2017 17:00:28 -0700
Subject: 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 | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

(limited to 'java')

diff --git a/java/com/android/dialer/dialpadview/DialpadKeyButton.java b/java/com/android/dialer/dialpadview/DialpadKeyButton.java
index 24ca9cc86..0c842ebb5 100644
--- a/java/com/android/dialer/dialpadview/DialpadKeyButton.java
+++ b/java/com/android/dialer/dialpadview/DialpadKeyButton.java
@@ -167,17 +167,15 @@ public class DialpadKeyButton extends FrameLayout {
           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;
       }
     }
 
-- 
cgit v1.2.3