summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid Trainor <dtrainor@chromium.org>2014-09-02 11:51:09 -0700
committerDavid Trainor <dtrainor@chromium.org>2014-09-02 18:57:52 +0000
commitb1838a4e4c2d0a1b9ca78d3f40ee33bcbf0670b5 (patch)
tree00fa8faf777bc09e582df6d42c4a192b66a1014d
parent6debe587e7d6c302b98f287d1b090a50a97025e3 (diff)
downloadchromium_src-b1838a4e4c2d0a1b9ca78d3f40ee33bcbf0670b5.zip
chromium_src-b1838a4e4c2d0a1b9ca78d3f40ee33bcbf0670b5.tar.gz
chromium_src-b1838a4e4c2d0a1b9ca78d3f40ee33bcbf0670b5.tar.bz2
[Android] Fix overflow menu enter tablet animation
- On tablets the new anchor point was 6dp off due to the new shadow assets. To fix this add a 6dp translate to the enter animation that will move the asset into the correct location as the scale animation runs. This effectively mimics the old animation but allows the offset to be included. BUG=405352 Review URL: https://codereview.chromium.org/488863002 Cr-Commit-Position: refs/heads/master@{#290773} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@290773 0039d316-1c4b-4281-b951-d872f2087c98 (cherry picked from commit 6f2a9c8524cf5a0e92bffeb63550be0321bacfbb) Review URL: https://codereview.chromium.org/533813002 Cr-Commit-Position: refs/branch-heads/2125@{#180} Cr-Branched-From: b68026d94bda36dd106a3d91a098719f952a9477-refs/heads/master@{#290040}
-rw-r--r--chrome/android/java/res/anim/menu_enter.xml4
-rw-r--r--chrome/android/java/res/values-sw600dp/dimens.xml2
-rw-r--r--chrome/android/java/res/values/dimens.xml2
-rw-r--r--chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenu.java8
4 files changed, 10 insertions, 6 deletions
diff --git a/chrome/android/java/res/anim/menu_enter.xml b/chrome/android/java/res/anim/menu_enter.xml
index 63194a9..e6c4936 100644
--- a/chrome/android/java/res/anim/menu_enter.xml
+++ b/chrome/android/java/res/anim/menu_enter.xml
@@ -16,4 +16,8 @@
<alpha android:interpolator="@android:anim/linear_interpolator"
android:fromAlpha="0" android:toAlpha="1"
android:duration="200" />
+ <translate android:interpolator="@interpolator/transform_curve_interpolator"
+ android:fromYDelta="@dimen/menu_negative_software_vertical_offset"
+ android:toYDelta="0"
+ android:duration="200" />
</set> \ No newline at end of file
diff --git a/chrome/android/java/res/values-sw600dp/dimens.xml b/chrome/android/java/res/values-sw600dp/dimens.xml
index d8fed33..ba5395e 100644
--- a/chrome/android/java/res/values-sw600dp/dimens.xml
+++ b/chrome/android/java/res/values-sw600dp/dimens.xml
@@ -7,5 +7,5 @@
<resources>
<!-- Menu Dimensions -->
<!-- Necessary to align the menu icon with the actual button. -->
- <dimen name="menu_software_vertical_offset">-6dp</dimen>
+ <dimen name="menu_negative_software_vertical_offset">6dp</dimen>
</resources> \ No newline at end of file
diff --git a/chrome/android/java/res/values/dimens.xml b/chrome/android/java/res/values/dimens.xml
index 72172f2..fa87b13 100644
--- a/chrome/android/java/res/values/dimens.xml
+++ b/chrome/android/java/res/values/dimens.xml
@@ -17,7 +17,7 @@
<!-- Custom Menu dimensions -->
<dimen name="menu_width">258dp</dimen>
- <dimen name="menu_software_vertical_offset">0dp</dimen>
+ <dimen name="menu_negative_software_vertical_offset">0dp</dimen>
<!-- The amount to fade the edges of the menu to indicate more content is available
via scrolling. -->
<dimen name="menu_vertical_fade_distance">15dp</dimen>
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenu.java b/chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenu.java
index af48e84..b3d1cfe 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenu.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenu.java
@@ -46,7 +46,7 @@ public class AppMenu implements OnItemClickListener, OnKeyListener {
private final int mItemRowHeight;
private final int mItemDividerHeight;
private final int mVerticalFadeDistance;
- private final int mAdditionalVerticalOffset;
+ private final int mNegativeSoftwareVerticalOffset;
private ListPopupWindow mPopup;
private AppMenuAdapter mAdapter;
private AppMenuHandler mHandler;
@@ -73,8 +73,8 @@ public class AppMenu implements OnItemClickListener, OnKeyListener {
mItemDividerHeight = itemDividerHeight;
assert mItemDividerHeight >= 0;
- mAdditionalVerticalOffset =
- res.getDimensionPixelSize(R.dimen.menu_software_vertical_offset);
+ mNegativeSoftwareVerticalOffset =
+ res.getDimensionPixelSize(R.dimen.menu_negative_software_vertical_offset);
mVerticalFadeDistance = res.getDimensionPixelSize(R.dimen.menu_vertical_fade_distance);
}
@@ -207,7 +207,7 @@ public class AppMenu implements OnItemClickListener, OnKeyListener {
} else {
// The menu is displayed over and below the anchored view, so shift the menu up by the
// height of the anchor view.
- popup.setVerticalOffset(mAdditionalVerticalOffset - anchorHeight);
+ popup.setVerticalOffset(-mNegativeSoftwareVerticalOffset - anchorHeight);
}
}