diff options
author | Romain Guy <romainguy@google.com> | 2010-04-05 18:39:24 -0700 |
---|---|---|
committer | Romain Guy <romainguy@google.com> | 2010-04-05 18:39:24 -0700 |
commit | 3af600182f200b52ac93897061efb4896dd3ea8e (patch) | |
tree | 2f77a6d8464be07d5707cebb921fc4955b9712da | |
parent | 327a9a3a309eeda5bdc18281066f2e19236455bc (diff) | |
download | android_packages_apps_Trebuchet-3af600182f200b52ac93897061efb4896dd3ea8e.tar.gz android_packages_apps_Trebuchet-3af600182f200b52ac93897061efb4896dd3ea8e.tar.bz2 android_packages_apps_Trebuchet-3af600182f200b52ac93897061efb4896dd3ea8e.zip |
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
-rw-r--r-- | proguard.flags | 4 | ||||
-rw-r--r-- | res/layout-land/launcher.xml | 8 | ||||
-rw-r--r-- | res/values/attrs.xml | 4 | ||||
-rw-r--r-- | src/com/android/launcher2/ClippedImageView.java | 50 |
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); + + } +} |