commit 330a6346484929afea3e880ee0f30415fb7756ab
author Matthew Ng <ngmatthew@google.com> Tue Jun 19 15:32:56 2018 -0700
committer Matthew Ng <ngmatthew@google.com> Wed Jun 20 12:54:36 2018 -0700
tree 93d5b7eb7e14a22ecdfac252dae6f20c0e63c1e5
parent 3471127e8b8e53ce1e6f4d499455e395892dfb9d

Allow clear all button in overview be clickable if visible

Test: enter overview, make sure clear all button is invisible, try to
tap its location
Fixes: 110432640

Change-Id: I7531e4c6cfdcc9a29d98da24a73c32b8080e013c

quickstep/src/com/android/quickstep/views/ClearAllButton.java

diff --git a/quickstep/src/com/android/quickstep/views/ClearAllButton.java b/quickstep/src/com/android/quickstep/views/ClearAllButton.java
index 3911931..fbecd84 100644
--- a/quickstep/src/com/android/quickstep/views/ClearAllButton.java
+++ b/quickstep/src/com/android/quickstep/views/ClearAllButton.java
@@ -54,7 +54,7 @@
     public void setContentAlpha(float alpha) {
         if (mContentAlpha != alpha) {
             mContentAlpha = alpha;
-            setAlpha(mScrollAlpha * mContentAlpha);
+            updateAlpha();
         }
     }
 
@@ -68,6 +68,12 @@
         float shift = Math.min(scrollState.scrollFromEdge, width);
         setTranslationX(mIsRtl ? (mScrollOffset - shift) : (mScrollOffset + shift));
         mScrollAlpha = 1 - shift / width;
-        setAlpha(mScrollAlpha * mContentAlpha);
+        updateAlpha();
+    }
+
+    private void updateAlpha() {
+        final float alpha = mScrollAlpha * mContentAlpha;
+        setAlpha(alpha);
+        setClickable(alpha == 1);
     }
 }