From 59ffaf035d2f9a1273072483004d9a4a0abbb5b9 Mon Sep 17 00:00:00 2001 From: Matthew Ng Date: Tue, 12 Jun 2018 12:57:29 -0700 Subject: [PATCH] Fixes the task header view location in landscape The header was off by forgetting to calculate with insets. Also rtl needed to start from left to do calculations correctly. Test: start overview in landscape, press icon of task Change-Id: I561dbe315c7be0b4dee602bfd17b4abfffa55605 Fixes: 110096772 --- quickstep/src/com/android/quickstep/views/TaskMenuView.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/quickstep/src/com/android/quickstep/views/TaskMenuView.java b/quickstep/src/com/android/quickstep/views/TaskMenuView.java index 39f532320d..6eb685479b 100644 --- a/quickstep/src/com/android/quickstep/views/TaskMenuView.java +++ b/quickstep/src/com/android/quickstep/views/TaskMenuView.java @@ -26,6 +26,7 @@ import android.content.res.Configuration; import android.graphics.Rect; import android.graphics.drawable.Drawable; import android.util.AttributeSet; +import android.view.Gravity; import android.view.MotionEvent; import android.view.ViewGroup; import android.widget.LinearLayout; @@ -172,8 +173,9 @@ public class TaskMenuView extends AbstractFloatingView { Rect insets = mActivity.getDragLayer().getInsets(); BaseDragLayer.LayoutParams params = (BaseDragLayer.LayoutParams) getLayoutParams(); params.width = sTempRect.width(); + params.gravity = Gravity.LEFT; setLayoutParams(params); - setX(Utilities.isRtl(getResources()) ? -sTempRect.left : sTempRect.left); + setX(sTempRect.left - insets.left); setY(sTempRect.top + getResources().getDimension(R.dimen.task_thumbnail_top_margin) - insets.top); }