From 7744169812e0d35531e9de258785af1f2ec9bed3 Mon Sep 17 00:00:00 2001 From: Hyunyoung Song Date: Mon, 27 Jun 2016 22:00:48 -0700 Subject: [PATCH] Extend all apps pull up activation area to page indicator view b/29752599 Change-Id: I3ee5efefa98defbc8f386895753946efe79708b4 Background: page indicator's background is where the caret symbol is. --- src/com/android/launcher3/PagedView.java | 2 +- .../launcher3/allapps/AllAppsTransitionController.java | 4 +++- src/com/android/launcher3/dragndrop/DragLayer.java | 5 +++++ 3 files changed, 9 insertions(+), 2 deletions(-) diff --git a/src/com/android/launcher3/PagedView.java b/src/com/android/launcher3/PagedView.java index 5ac3f0bab9..2d3e8dd04e 100644 --- a/src/com/android/launcher3/PagedView.java +++ b/src/com/android/launcher3/PagedView.java @@ -321,7 +321,7 @@ public abstract class PagedView extends ViewGroup implements ViewGroup.OnHierarc return (getMeasuredHeight() - getViewportHeight()) / 2; } - PageIndicator getPageIndicator() { + public PageIndicator getPageIndicator() { return mPageIndicator; } diff --git a/src/com/android/launcher3/allapps/AllAppsTransitionController.java b/src/com/android/launcher3/allapps/AllAppsTransitionController.java index 3157c133f2..585c5206d3 100644 --- a/src/com/android/launcher3/allapps/AllAppsTransitionController.java +++ b/src/com/android/launcher3/allapps/AllAppsTransitionController.java @@ -137,7 +137,9 @@ public class AllAppsTransitionController implements TouchController, VerticalPul return true; } } else { - if (mLauncher.getDragLayer().isEventOverHotseat(ev) && !grid.isVerticalBarLayout()) { + if ((mLauncher.getDragLayer().isEventOverHotseat(ev) + || mLauncher.getDragLayer().isEventOverPageIndicator(ev)) + && !grid.isVerticalBarLayout()) { return true; } } diff --git a/src/com/android/launcher3/dragndrop/DragLayer.java b/src/com/android/launcher3/dragndrop/DragLayer.java index 4966938353..8aed6d8349 100644 --- a/src/com/android/launcher3/dragndrop/DragLayer.java +++ b/src/com/android/launcher3/dragndrop/DragLayer.java @@ -187,6 +187,11 @@ public class DragLayer extends InsettableFrameLayout { removeView(mOverlayView); } + public boolean isEventOverPageIndicator(MotionEvent ev) { + getDescendantRectRelativeToSelf(mLauncher.getWorkspace().getPageIndicator(), mHitRect); + return mHitRect.contains((int) ev.getX(), (int) ev.getY()); + } + public boolean isEventOverHotseat(MotionEvent ev) { getDescendantRectRelativeToSelf(mLauncher.getHotseat(), mHitRect); return mHitRect.contains((int) ev.getX(), (int) ev.getY());