Last minute hack to deliver touch events to widget in landscape mode.

Bug #2562729

In landscape, the left and right dots buttons are 93dip wide; this makes them overlap
with widgets at the bottom of the screen. The dots do not need to be that wide but
we chose this size to make it easier to tap them in portrait. To avoid issues in
landscape, this change introduces a new type of ImageView that can ignore touch
events in a certain zone. This was easier and cheaper than re-cutting 36+ assets.

Change-Id: Id261fba41a43dede943e72060e39e87658e4b0df

4 files changed, 64 insertions, 2 deletions

diff --git a/proguard.flags b/proguard.flags
index 5d72ef32b..255f97050 100644
--- a/proguard.flags
+++ b/proguard.flags
@@ -6,3 +6,7 @@
 -keep class com.android.launcher2.AllApps3D$Defines {
   *;
 }
+
+-keep class com.android.launcher2.ClippedImageView {
+  *;
+}
diff --git a/res/layout-land/launcher.xml b/res/layout-land/launcher.xml
index 61ebad0fb..44e1cfb55 100644
--- a/res/layout-land/launcher.xml
+++ b/res/layout-land/launcher.xml
@@ -41,7 +41,7 @@
 
     </com.android.launcher2.Workspace>
 
-    <ImageView
+    <com.android.launcher2.ClippedImageView
         android:id="@+id/previous_screen"
         android:layout_width="93dip"
         android:layout_height="@dimen/button_bar_height"
@@ -53,10 +53,12 @@
         
         android:onClick="previousScreen"
 
+        launcher:ignoreZone="56dip"
+
         android:focusable="true"
         android:clickable="true" />
 
-    <ImageView
+    <com.android.launcher2.ClippedImageView
         android:id="@+id/next_screen"
         android:layout_width="93dip"
         android:layout_height="@dimen/button_bar_height"
@@ -68,6 +70,8 @@
         
         android:onClick="nextScreen"
         
+        launcher:ignoreZone="-56dip"
+        
         android:focusable="true"
         android:clickable="true" />
 
diff --git a/res/values/attrs.xml b/res/values/attrs.xml
index a0085bc40..78391206e 100644
--- a/res/values/attrs.xml
+++ b/res/values/attrs.xml
@@ -82,4 +82,8 @@
         <attr name="title" format="reference" />
         <attr name="uri" format="string" />
     </declare-styleable>
+
+    <declare-styleable name="ClippedImageView">
+        <attr name="ignoreZone" format="dimension" />
+    </declare-styleable>
 </resources>
diff --git a/src/com/android/launcher2/ClippedImageView.java b/src/com/android/launcher2/ClippedImageView.java
new file mode 100644
index 000000000..2c4380d88
--- /dev/null
+++ b/src/com/android/launcher2/ClippedImageView.java
@@ -0,0 +1,50 @@
+/*
+ * Copyright (C) 2010 The Android Open Source Project
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package com.android.launcher2;
+
+import android.content.Context;
+import android.content.res.TypedArray;
+import android.util.AttributeSet;
+import android.view.MotionEvent;
+import android.widget.ImageView;
+import com.android.launcher.R;
+
+public class ClippedImageView extends ImageView {
+    private final int mZone;
+
+    public ClippedImageView(Context context, AttributeSet attrs) {
+        this(context, attrs, 0);
+    }
+
+    public ClippedImageView(Context context, AttributeSet attrs, int defStyle) {
+        super(context, attrs, defStyle);
+        
+        TypedArray a = context.obtainStyledAttributes(attrs, R.styleable.ClippedImageView, defStyle, 0);
+
+        mZone = a.getDimensionPixelSize(R.styleable.ClippedImageView_ignoreZone, 0);
+
+        a.recycle();
+    }
+
+    @Override
+    public boolean onTouchEvent(MotionEvent event) {
+        final int zone = mZone;
+        return !(zone != 0 && (zone > 0 && event.getX() >= zone) ||
+                (zone < 0 && event.getX() < getWidth() + zone)) && super.onTouchEvent(event);
+
+    }
+}