diff --git a/quickstep/src/com/android/quickstep/util/RecentsViewUtils.kt b/quickstep/src/com/android/quickstep/util/RecentsViewUtils.kt index be1af64f02..e70372fb79 100644 --- a/quickstep/src/com/android/quickstep/util/RecentsViewUtils.kt +++ b/quickstep/src/com/android/quickstep/util/RecentsViewUtils.kt @@ -66,6 +66,9 @@ class RecentsViewUtils { fun getLargeTaskViewIds(taskViews: Iterable): List = taskViews.filter { it.isLargeTile }.map { it.taskViewId } + /** Counts [TaskView]s that are large tiles. */ + fun getLargeTileCount(taskViews: Iterable): Int = taskViews.count { it.isLargeTile } + /** * Returns the first TaskView that should be displayed as a large tile. * diff --git a/quickstep/src/com/android/quickstep/views/RecentsView.java b/quickstep/src/com/android/quickstep/views/RecentsView.java index 3cd95b0304..8b2c95de5c 100644 --- a/quickstep/src/com/android/quickstep/views/RecentsView.java +++ b/quickstep/src/com/android/quickstep/views/RecentsView.java @@ -3669,8 +3669,8 @@ public abstract class RecentsView< float longGridRowWidthDiff = 0; int topGridRowSize = mTopRowIdSet.size(); - int bottomGridRowSize = taskCount - mTopRowIdSet.size() - - (enableGridOnlyOverview() ? 0 : 1); + int numLargeTiles = mUtils.getLargeTileCount(getTaskViews()); + int bottomGridRowSize = taskCount - mTopRowIdSet.size() - numLargeTiles; boolean topRowLonger = topGridRowSize > bottomGridRowSize; boolean bottomRowLonger = bottomGridRowSize > topGridRowSize; boolean dismissedTaskFromTop = mTopRowIdSet.contains(dismissedTaskViewId); @@ -3969,12 +3969,12 @@ public abstract class RecentsView< final boolean finalCloseGapBetweenClearAll = closeGapBetweenClearAll; final boolean finalSnapToLastTask = snapToLastTask; final boolean finalIsFocusedTaskDismissed = isFocusedTaskDismissed; - mPendingAnimation.addEndListener(new Consumer() { + mPendingAnimation.addEndListener(new Consumer<>() { @Override public void accept(Boolean success) { if (mEnableDrawingLiveTile && dismissedTaskView.isRunningTask() && success) { finishRecentsAnimation(true /* toRecents */, false /* shouldPip */, - () -> onEnd(success)); + () -> onEnd(true)); } else { onEnd(success); } @@ -4144,6 +4144,14 @@ public abstract class RecentsView< // If snapping to last task, find the last task after dismissal. pageToSnapTo = indexOfChild( getLastGridTaskView(topRowIdArray, bottomRowIdArray)); + + if (pageToSnapTo == INVALID_PAGE) { + // Snap to latest large tile page after dismissing the + // last grid task. This will prevent snapping to page 0 when + // desktop task is visible as large tile. + pageToSnapTo = indexOfChild( + mUtils.getLastLargeTaskView(getTaskViews())); + } } else if (taskViewIdToSnapTo != -1) { // If snapping to another page due to indices rearranging, find // the new index after dismissal & rearrange using the task view id.