From f679ba942fab1ed48f0d89528f11941f85c45a21 Mon Sep 17 00:00:00 2001 From: mpodolian Date: Tue, 22 Oct 2024 10:08:25 -0700 Subject: [PATCH] Removed stashing hotseat calls. Removed calls that were stashing and unstashing the hotseat bar. Flag: com.android.wm.shell.enable_bubble_bar Test: Manual. Be on Launcher home, have some bubbles. Expand / collapse bubble bar. Hotseat remains behind the scream view. Fixes: 373429249 Change-Id: I1a9ba39c8ca11d49340f217f095d5e872b06105b --- .../launcher3/taskbar/TaskbarStashController.java | 12 ++++++++++-- .../taskbar/bubbles/BubbleBarViewController.java | 12 +++--------- 2 files changed, 13 insertions(+), 11 deletions(-) diff --git a/quickstep/src/com/android/launcher3/taskbar/TaskbarStashController.java b/quickstep/src/com/android/launcher3/taskbar/TaskbarStashController.java index b19da6b8e2..c1dd216a9c 100644 --- a/quickstep/src/com/android/launcher3/taskbar/TaskbarStashController.java +++ b/quickstep/src/com/android/launcher3/taskbar/TaskbarStashController.java @@ -410,12 +410,20 @@ public class TaskbarStashController implements TaskbarControllers.LoggableTaskba return mIsStashed; } - /** Sets the hotseat stashed. */ + /** + * Sets the hotseat stashed. + * b/373429249 - we might change this behavior if we remove the scrim, that's why we're keeping + * this method + */ public void stashHotseat(boolean stash) { mControllers.uiController.stashHotseat(stash); } - /** Instantly un-stashes the hotseat. */ + /** + * Instantly un-stashes the hotseat. + * * b/373429249 - we might change this behavior if we remove the scrim, that's why we're + * keeping this method + */ public void unStashHotseatInstantly() { mControllers.uiController.unStashHotseatInstantly(); } diff --git a/quickstep/src/com/android/launcher3/taskbar/bubbles/BubbleBarViewController.java b/quickstep/src/com/android/launcher3/taskbar/bubbles/BubbleBarViewController.java index 76d36061f1..ec2c7574e4 100644 --- a/quickstep/src/com/android/launcher3/taskbar/bubbles/BubbleBarViewController.java +++ b/quickstep/src/com/android/launcher3/taskbar/bubbles/BubbleBarViewController.java @@ -843,23 +843,17 @@ public class BubbleBarViewController { /** * Hides the persistent taskbar if it is going to intersect with the expanded bubble bar if in - * app or overview. Set the hotseat stashed state if on launcher home screen. If not on launcher - * home screen and hotseat is stashed immediately un-stashes the hotseat. + * app or overview. */ private void adjustTaskbarAndHotseatToBubbleBarState(boolean isBubbleBarExpanded) { - if (mBubbleStashController.isBubblesShowingOnHome()) { - mTaskbarStashController.stashHotseat(isBubbleBarExpanded); - } else if (!mBubbleStashController.isTransientTaskBar()) { + if (!mBubbleStashController.isBubblesShowingOnHome() + && !mBubbleStashController.isTransientTaskBar()) { boolean hideTaskbar = isBubbleBarExpanded && isIntersectingTaskbar(); mTaskbarViewPropertiesProvider .getIconsAlpha() .animateToValue(hideTaskbar ? 0 : 1) .start(); } - if (!mBubbleStashController.isBubblesShowingOnHome() - && mTaskbarStashController.isHiddenForBubbles()) { - mTaskbarStashController.unStashHotseatInstantly(); - } } /** Return {@code true} if expanded bubble bar would intersect the taskbar. */