diff options
author | David Trainor <dtrainor@chromium.org> | 2014-09-02 11:51:09 -0700 |
---|---|---|
committer | David Trainor <dtrainor@chromium.org> | 2014-09-02 18:57:52 +0000 |
commit | b1838a4e4c2d0a1b9ca78d3f40ee33bcbf0670b5 (patch) | |
tree | 00fa8faf777bc09e582df6d42c4a192b66a1014d | |
parent | 6debe587e7d6c302b98f287d1b090a50a97025e3 (diff) | |
download | chromium_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}
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); } } |