Commit Graph

12955 Commits

Author SHA1 Message Date
Sukesh Ram
3cecfd077c Fix Taskbar 3 Button y position on launcher home pause/resume
When app launcher pauses (not on app entry, but whilst home), we want to move the taskbar buttons to 3-button mode to avoid overlap with any popup UI (usually in the form of bottom sheets). To avoid overriding existing app entry animations, this is only implemented for cases of launcher pause without a launcher stop.

Flag: EXEMPT bugfix
Bug: 373977342
Bug: 374058222
Test: Manual
Change-Id: I727173b0ae7f3f8e23b1da6755ceda6493b311cb
2024-11-28 00:27:42 +00:00
Mykola Podolian
400fc4ad98 Merge changes I239f9357,I57a32ac5 into main
* changes:
  Update setting taskbar window height
  Set bubble bar aligned to QSB
2024-11-27 20:44:53 +00:00
Uwais Ashraf
bfede7eb48 Merge "Exclude cleanup methods from CUJ for entering split select." into main 2024-11-27 20:28:40 +00:00
Treehugger Robot
d7b3369036 Merge "Ignore events that occur between app icons on the taskbar" into main 2024-11-27 20:22:17 +00:00
Liran Binyamin
ed249f0451 Merge "Show the bubble bar edu view" into main 2024-11-27 20:09:07 +00:00
mpodolian
3111daf2f1 Update setting taskbar window height
Updated the logic to set the taskbar window height to accommodate the
bubble bar with its maximum flyout height before the bubble bar
animation begins. The height is reverted to the default value once the
animation is completed.

Fixes: 378400160
Flag: com.android.wm.shell.enable_bubble_bar
Test: Manual. Set 3-button navigation mode and have the QSB placed
above the hotseat. Send a notification that will display a bubble, and
observe the bubble bar animation and final placement aligned with the
QSB.
Next, set gesture navigation mode and repeat the flow above. Observe
that the bubble bar is aligned with the hotseat icons.
Felix video:
http://recall/-/gx8ASgewUeUS3QYohfrd1J/gYMVt0sH4si0KOpaBxIPem
Tangor video:
http://recall/-/gx8ASgewUeUS3QYohfrd1J/fz95MbjcjW3awStnuOl8BJ

Change-Id: I239f935743c3936fed44822c3c62652073ffefe2
2024-11-27 10:19:24 -08:00
mpodolian
28907bcbd6 Set bubble bar aligned to QSB
Change logic to updated bubble bar alignment with the QSB vertical
center for 3 buttons navigation mode.

Fixes: 378400160
Flag: com.android.wm.shell.enable_bubble_bar
Test: PersistentBubbleStashControllerTest
Test: TransientBubbleStashControllerTest
Test: Manual. Set the buttons navigation mode and observe that the
bubble bar is centered with the QSB when the QSB is not aligned with the
hotseat. Then, switch to gestures navigation mode and observe that the
bubble bar is centered with the hotseat when the QSB is not aligned with
the hotseat.

Change-Id: I57a32ac5bde7a7055b54a89727429424e5c0ca4f
2024-11-27 10:15:08 -08:00
Artsiom Mitrokhin
75974d8361 Merge "Prevent spawning multiple pinning popups on right click" into main 2024-11-27 15:49:19 +00:00
Marcelo Arteiro
8b63ee3420 Merge "Migrate materialColor* attributes into colors" into main 2024-11-27 14:55:47 +00:00
Vinay Joglekar
0bb2421c94 Merge "Fix animation while hiding Desktop tasks during split" into main 2024-11-27 13:33:59 +00:00
Treehugger Robot
02038f4deb Merge "Log various TaskView properties in TaksContainer.itemInfo" into main 2024-11-27 12:59:31 +00:00
vinayjoglekar
f567e7e2fa Fix animation while hiding Desktop tasks during split
1. Scale down desktop tasks
2. Change translation start when all desktop tasks need to hidden

Fix: 378447894
Flag: com.android.launcher3.enable_large_desktop_windowing_tile
Test: Manual: Add desktop window and split focussed task, desktop task(s)(also in RTL)
Change-Id: I25c11d654539efc6529a2e995d8d00cbf73c870d
2024-11-27 11:34:22 +00:00
Alex Chau
09f2433253 Log various TaskView properties in TaksContainer.itemInfo
- Extracted itemInfo creation in to TaskViewItemInfo and added a test
- Created a standalone atom for TaskView, logging data we want (please refer to the bug)
- Convert TaskView atom to StatsLog data in StatsLogCompatManager
- Simplified DesktopSystemShortcutTest and ExternalDisplaySystemShortcutTest to avoid extra testing on DesktopModeStatus

Fix: 341320349
Test: TaskViewItemInfoTest, DesktopSystemShortcutTest, ExternalDisplaySystemShortcutTest
Flag: EXEMPT logging
Change-Id: Icf04ff2b1670c3f3d5f6a66f99c0d28314efaf21
2024-11-27 10:41:43 +00:00
Uwais Ashraf
02f7a1a96d Exclude cleanup methods from CUJ for entering split select.
Fix: 378007383
Flag: EXEMPT BUG_FIX
Test: ABTD run - https://android-build.corp.google.com/builds/abtd/run/L73400030008027437
Change-Id: Ied6897a8a84b4411517ac4fd9904e00eeb86d3ad
2024-11-27 10:05:48 +00:00
Treehugger Robot
a8f25d26ad Merge "[Launcher Memory Leak] Avoid leaking Folder/FolderIcon when removing FolderIcon from TaskbarView" into main 2024-11-26 20:37:21 +00:00
Uwais Ashraf
322b831d1d Merge "Make RV focusable when empty to read out content desc for TalkBack." into main 2024-11-26 15:25:25 +00:00
Liran Binyamin
16a604fe0c Show the bubble bar edu view
We now show the bubble bar edu view after a request from shell
to show it was received, and as soon as the user performs a touch
gesture on the bubble bar or flyout that causes the bar to be
expanded.

The click listener on the bubble bar is removed with this change
because the input consumer already handles taps. Removing the
listener avoids processing the tap twice which causes the edu view
to not be displayed. Removing the click listener solves the issue
but ideally we should move the edu view to launcher since that
would allow us to manage its state easily.

Flag: com.android.wm.shell.enable_bubble_bar
Fixes: 374842575
Test: atest PersistentBubbleStashControllerTest
Test: atest BubbleBarSwipeControllerTest
Test: atest TransientBubbleStashControllerTest
Test: manual
       - force show the bar edu view
       - send a bubble
       - tap the bubble bar to expand
       - observe bubble bar edu is visible
       - repeat with swiping on the bar and tapping the flyout
       - launch an app
       - swipe up to show the taskbar
       - observe edu view not visible
       - send a bubble with auto expand
       - observe bubble bar expanded and edu view not visible
Change-Id: I98ec44e6edf6c69d6d47b5709140796c2ecab23e
2024-11-26 10:23:45 -05:00
Uwais Ashraf
6c85912736 Make RV focusable when empty to read out content desc for TalkBack.
Fix: 379638096
Flag: EXEMPT BUG_FIX
Test: Manual - go to overview with and without tasks. Check TalkBack.
Change-Id: I1f3faace530ed3c9b0384b94dbc163f5b8c15dda
2024-11-26 11:42:42 +00:00
Chaitanya Cheemala (xWF)
1743fd9e77 Merge "Revert "Use the Coreographer's frame time for a more reliable ti..."" into main 2024-11-26 11:37:15 +00:00
Chaitanya Cheemala (xWF)
9155cf9066 Revert "Use the Coreographer's frame time for a more reliable ti..."
Revert submission 30509816-raf-timing

Reason for revert: Likely culprit for b/381006945  - verified through ABTD before revert submission as part of standard investigation process.

Reverted changes: /q/submissionid:30509816-raf-timing

Change-Id: Iaed968120be662df8d915bac61e7cfa5b936f35c
2024-11-26 11:35:00 +00:00
Alex Chau
1d720793fa Merge "Add TaskOverlay children for accessibility" into main 2024-11-26 10:49:34 +00:00
Treehugger Robot
cff3ade324 Merge "Remove padding from All Apps and Divider views for transient taskbar." into main 2024-11-26 04:02:56 +00:00
Treehugger Robot
f612dc1a5b Merge "Fix taskbar visibility when default-to-desktop" into main 2024-11-25 23:18:43 +00:00
Fengjiang Li
3daaad8498 [Launcher Memory Leak] Avoid leaking Folder/FolderIcon when removing FolderIcon from TaskbarView
Fix: 380868830
Flag: NONE - released code
Test: Presubmit
Change-Id: I57c179384df06015a7e95a173c399cfcda236e46
2024-11-25 11:30:53 -08:00
Alex Chau
0aee46e098 Add TaskOverlay children for accessibility
- ag/28947579 add a speicifc order for children of TaskView to be added for accessibility. All TaskView children will need to added through TaskContainer.addChildForAccessibility

Fix: 379802031
Fix: 380016834
Fix: 204280593
Test: Go to Select mode, ensures Talkback can cycle through text selection boxes
Flag: EXEMPT BUG_FIX
Change-Id: Id7991a8b3e76fe47f562e338e9967c8e85fa4908
2024-11-25 19:04:46 +00:00
Saumya Prakash
43ecc1d0a1 Merge "Fix Taskbar not auto stashing from multi instance menu" into main 2024-11-25 18:34:35 +00:00
Jon Miranda
f86e0e4b88 Remove padding from All Apps and Divider views for transient taskbar.
(Initial CL removed padding always) I0f9cce376d47918e69a65ed288f10143930ee3be

Bug: 375661139
Test: confirmed with design team
Flag: EXEMPT bugfix
Change-Id: I467d0a5ae3ab48ebebc5a05f7a0902ddd01f8084
2024-11-25 18:16:26 +00:00
Schneider Victor-tulias
54e50f43a6 Merge "Start the home intent when swiping from home to home" into main 2024-11-25 17:53:27 +00:00
Luca Zuccarini
fb2525a850 Merge "Use the Coreographer's frame time for a more reliable timestamp." into main 2024-11-25 16:19:35 +00:00
Schneider Victor-Tulias
ccd08f4667 Start the home intent when swiping from home to home
This allows us to clean up the home screen properly (close all apps and floating views)

Flag: com.android.launcher3.enable_fallback_overview_in_window
Fixes: 380049465
Fixes: 380049942
Test: TAPL tests
Test: swipe home from home all apps, with and without an open popup menu, with and without a 3P launcher
Change-Id: I3c3c8e832329afc3ffaa1de7cf646be79d84e9a3
2024-11-25 10:56:54 -05:00
Artsiom Mitrokhin
86bf6ae902 Prevent spawning multiple pinning popups on right click
The dismiss logic only checks for `ACTION_DOWN` events, but when the
user frequently right-clicks OR right-press-move-release, we get
multiple `ACTION_MOVE` events instead, which are ignored by the dismiss
logic of the previous popup instance.

Attached screen recordings to the bug.

Bug: 362475367
Flag: EXEMPT bugfix
Test: manual
Change-Id: I665c3127e4b449a8f4c71e443cc6ae07c1bf2794
2024-11-25 10:52:36 -05:00
Johannes Gallmann
0b0452eda9 Merge "Cancel predictive back when sliding off back button" into main 2024-11-25 08:17:56 +00:00
Johannes Gallmann
4af733d65f Merge "Only register Launcher back-callback when ROLE_HOME is held." into main 2024-11-25 08:13:05 +00:00
Dan Brunwasser
7005aff0dc Merge "Fix home gesture animation for 3rd party launchers." into main 2024-11-23 01:06:42 +00:00
Treehugger Robot
ba13bc11c6 Merge "[CD Taskbar] Refactor TaskbarManager to store TaskbarRootLayouts in a map" into main 2024-11-23 00:38:44 +00:00
Sukesh Ram
7516dd06d5 [CD Taskbar] Refactor TaskbarManager to store TaskbarRootLayouts in a map
Refactor the TaskbarManager's existing usages of TaskbarRootLayouts to use a map, so that there can be a 1:1 relationship between TaskbarRootLayouts and TaskbarActivityContexts.

Flag: EXEMPT not adding new behavior
Bug: 380351840
Test: Manual
Change-Id: Ib4379a9b6d50b8bdd64b70d7b15ce798ebb42731
2024-11-22 14:30:49 -08:00
Winson Chung
bf84fe574e Merge "Update OHM regions whenever other regions change" into main 2024-11-22 21:23:21 +00:00
Dan Brunwasser
e026b261b3 Fix home gesture animation for 3rd party launchers.
Previously home animations for 3rd party launchers were being cancelled by the recently added onFinishComplete call in AbsSwipeUpHandler.onTasksAppeared. This was because FallbackSwipeHandler.handleTasksAppeared was returning false even though mActiveAnimationFactory.handleHomeTaskAppeared was true. However, only fixing this so it returned true instead would allow AbsSwipeUpHandler.onTasksAppeared to continue to the following conditions which would fail, so this change also converts FallbackSwipeHandler.handleTasksAppeared to override onTasksAppeared instead. This allows home tasks to be handled first and skip splash screen handling since it is not needed.

Flag: NONE bug fix for recently introduced issue
Fixes: 362900007
Test: set 3rd party launcher as default and use home navigation gesture
Change-Id: I88015bd3bdbad94bd4897411714ff3f52a782499
2024-11-22 13:00:33 -08:00
Saumya Prakash
0028bed674 Fix Taskbar not auto stashing from multi instance menu
This change addresses an issue where touching the bottom of the screen would result in the dismissal of the multi instance menu, but leave the taskbar in a state that it would not auto stash. This is because the view was being removed through another way leading to taskbar being in a bad state. Now when the view is removed, the taskbar is correctly reset.

Fix: 380304394
Bug: 315989246
Test: Manually launch the menu then tap the bottom of the screen. Ensure that taskbar eventually auto stashes.
Flag: com.android.launcher3.enable_multi_instance_menu_taskbar
Change-Id: Ie6bae803a42cf0bcd0ff6362473a63cb22a841eb
2024-11-22 20:50:46 +00:00
Brian Isganitis
39837267eb Merge changes Ie1523446,I0263f704 into main
* changes:
  Do not pre-add All Apps icon in phone mode.
  Fix LayoutTransition All Apps divider logic for RTL.
2024-11-22 20:14:37 +00:00
Matt Sziklay
5763a6d0b7 Merge "Move shared logic to new package." into main 2024-11-22 18:53:58 +00:00
Winson Chung
ce1a73829f Update OHM regions whenever other regions change
- Speculative fix, there are some scenarios where we update the swipe
  touch/assistant regions without updating the OHM regions

Bug: 362039326
Flag: EXEMPT bugfix
Test: Manual
Change-Id: If5c558623073e5185827ae17a162dacf322c74a4
2024-11-22 16:53:22 +00:00
Johannes Gallmann
0610c625e0 Only register Launcher back-callback when ROLE_HOME is held.
When using a 3P Launcher, i.e. when the ROLE_HOME is not held by QuickstepLauncher, there should be no back callback registered with SystemUI.

Bug: 374882166
Flag: EXEMPT bugfix
Test: Manual, i.e. verified back-callback registration updates in SystemUI and also verified that back-to-home navigations in Nova Launcher are responsive.
Change-Id: I03187f80a4c3c893a6529f0a22613f6fdfe7b7b3
2024-11-22 13:49:00 +00:00
Johannes Gallmann
b28ff34177 Cancel predictive back when sliding off back button
Bug: 373544911
Test: Manual, i.e. verified that sliding away from the back button (within the taskbar window) cancels the predictive back animation
Flag: com.android.window.flags.predictive_back_three_button_nav
Change-Id: I9a1448c7005211c87a8979261b13fe37ab64a5b2
2024-11-22 13:09:56 +00:00
Alex Chau
ba8bfc97b0 Merge "Swipe up from excludeFromRecents task should be shown left of desktop tasks" into main 2024-11-22 11:59:12 +00:00
Uwais Ashraf
61cff8b686 Move fully visible task logic to more centralised method.
This means that overlay is initialised in all cases, not just some. Also uses pre-existing method for `isTaskViewFullyVisible()`

Fix: 376264918
Test: Manual checking of flows in bug.
Flag: com.android.launcher3.enable_refactor_task_thumbnail
Change-Id: Ia898a2a869b79fb120a162fa5a3fe7df0fbf9800
2024-11-22 10:59:33 +00:00
Brian Isganitis
6c0cd2278d Do not pre-add All Apps icon in phone mode.
Flag: com.android.launcher3.taskbar_recents_layout_transition
Bug: 343521765
Test: go/testedequals
Change-Id: Ie152344613cf982183c4bb72b8dde138f4f74e37
2024-11-21 23:38:00 +00:00
Brian Isganitis
f04ae6b09f Fix LayoutTransition All Apps divider logic for RTL.
We were checking the wrong index (off by one), leading to a crash if the divider is already in the Taskbar at the correct index. However, we do need to add it after the expected index if the divider isn't there, otherwise it will push the icon that is at that position to the right of the divider (it should be to the left).

Flag: com.android.launcher3.taskbar_recents_layout_transition
Bug: 343521765
Bug: 368522274
Test: go/testedequals
Change-Id: I0263f704728286b53da5b45828784b20aaae31ce
2024-11-21 23:38:00 +00:00
Treehugger Robot
423c85a2eb Merge "Remove padding from All Apps and Divider views." into main 2024-11-21 22:07:40 +00:00
Artsiom Mitrokhin
7205b24938 Ignore events that occur between app icons on the taskbar
Bug: 297325541
Flag: com.android.launcher3.show_taskbar_pinning_popup_from_anywhere
Test: manual, flip the flag, long-click between taskbar items
Change-Id: If2ee00f31fcd1788bac2c2c0cd78ba1eada86115
2024-11-21 16:38:32 -05:00