diff options
author | Hyunyoung Song <hyunyoungs@google.com> | 2017-07-19 21:23:13 +0000 |
---|---|---|
committer | Android (Google) Code Review <android-gerrit@google.com> | 2017-07-19 21:23:13 +0000 |
commit | 60326ece2027793181dfda5c43a028dc0e15b1da (patch) | |
tree | a439700f0fa27a35465128961d3d0607c6ce8160 /src | |
parent | 4426b1d27f120236fc994743178e73e2647255bc (diff) | |
parent | 435a35dec32c24e353a01228f3deef1d5b85810c (diff) | |
download | android_packages_apps_Trebuchet-60326ece2027793181dfda5c43a028dc0e15b1da.tar.gz android_packages_apps_Trebuchet-60326ece2027793181dfda5c43a028dc0e15b1da.tar.bz2 android_packages_apps_Trebuchet-60326ece2027793181dfda5c43a028dc0e15b1da.zip |
Merge "PopupContainerWithArrow should align correctly in RTL mode Test: see attached screenshot in the bug" into ub-launcher3-dorval-polish
Diffstat (limited to 'src')
-rw-r--r-- | src/com/android/launcher3/popup/PopupContainerWithArrow.java | 35 |
1 files changed, 20 insertions, 15 deletions
diff --git a/src/com/android/launcher3/popup/PopupContainerWithArrow.java b/src/com/android/launcher3/popup/PopupContainerWithArrow.java index 3de9bad4c..f5f7ddb31 100644 --- a/src/com/android/launcher3/popup/PopupContainerWithArrow.java +++ b/src/com/android/launcher3/popup/PopupContainerWithArrow.java @@ -399,6 +399,24 @@ public class PopupContainerWithArrow extends AbstractFloatingView implements Dra openAnim.start(); } + @Override + protected void onLayout(boolean changed, int l, int t, int r, int b) { + super.onLayout(changed, l, t, r, b); + DragLayer dragLayer = mLauncher.getDragLayer(); + if (getTranslationX() + l < 0 || + getTranslationX() + l + getMeasuredWidth() > dragLayer.getWidth()) { + // If we are still off screen, center horizontally too. + mGravity |= Gravity.CENTER_HORIZONTAL; + } + + if (Gravity.isHorizontal(mGravity)) { + setX(dragLayer.getWidth() / 2 - getMeasuredWidth() / 2); + } + if (Gravity.isVertical(mGravity)) { + setY(dragLayer.getHeight() / 2 - getMeasuredHeight() / 2); + } + } + /** * Returns the point at which the center of the arrow merges with the first popup item. */ @@ -517,21 +535,8 @@ public class PopupContainerWithArrow extends AbstractFloatingView implements Dra mIsAboveIcon = true; } - if (x < dragLayer.getLeft() || x + width > dragLayer.getRight()) { - // If we are still off screen, center horizontally too. - mGravity |= Gravity.CENTER_HORIZONTAL; - } - - if (Gravity.isHorizontal(mGravity)) { - setX(dragLayer.getWidth() / 2 - getMeasuredWidth() / 2); - } else { - setX(x); - } - if (Gravity.isVertical(mGravity)) { - setY(dragLayer.getHeight() / 2 - getMeasuredHeight() / 2); - } else { - setY(y); - } + setX(x); + setY(y); } private boolean isAlignedWithStart() { |