Commit Graph

4550 Commits

Author SHA1 Message Date
TreeHugger Robot
44ac266686 Merge "Fix split select scroll range when TaskView aren't shifting" into sc-v2-dev am: 11579eea19 am: 21192886c4
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16015203

Change-Id: I91b37c5388f5e0eb7d2f53ea71ad8c8272fb6f09
2021-10-11 20:22:20 +00:00
TreeHugger Robot
11579eea19 Merge "Fix split select scroll range when TaskView aren't shifting" into sc-v2-dev 2021-10-11 19:53:24 +00:00
TreeHugger Robot
f56ccbeffc Merge "Add scroll logging to RecentsView behind DEBUG flag" into sc-v2-dev am: 1569754519 am: 98680d64f6
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16015197

Change-Id: I93df9f2d0fa7df501cde11b1beebcb63278d2082
2021-10-11 18:47:37 +00:00
TreeHugger Robot
1569754519 Merge "Add scroll logging to RecentsView behind DEBUG flag" into sc-v2-dev 2021-10-11 18:13:45 +00:00
Alex Chau
02150305f9 Fix split select scroll range when TaskView aren't shifting
- There is an edge case that when ClearAllButton is visible, no split translation is applied, the offset adjustment needs to be calculated differently in that case
- Also, apply the scrollOffset to the TaskViews / ClearAllButton instead of updating min/max scroll

Bug: 200537659
Test: Split right in grid, split left with clear all button, split left without clear all button
Change-Id: I869c448bbec6aec8fa070e47193a692be6f75e84
2021-10-11 19:02:28 +01:00
TreeHugger Robot
728bd6f4fe Merge changes I499c59bd,Ia7260c60,I90846e65 into sc-v2-dev am: 4e9a3d5bdf am: 3b4d624128
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16012847

Change-Id: I6251cd20346c098bbedc37d04ff2ca6b1b6e0ae1
2021-10-08 17:41:02 +00:00
Alex Chau
e95de31304 Add scroll logging to RecentsView behind DEBUG flag
Bug: 202514687
Test: manual
Change-Id: Iec022f2583baa794904242a44a2c8dca8ea4b1be
2021-10-08 16:26:08 +01:00
Tony Wickham
1bd4bf7ff1 When taskbar would be stashed in 3 button mode, hide icons instead
In 0 button mode, stashing morphs the icons into the home handle. In 3 button mode, that doesn't make sense, but we should still hide the icons (and just keep the 3 nav buttons). You still can't manually stash via long press in 3 button mode, but this will address other states where we automatically stash the taskbar when we want to hide the icons from users (e.g. when pinning an app).

Test: Pin an app from overview in 3 button mode, ensure the taskbar icons disappear until unpinning
Bug: 190192993
Change-Id: I499c59bd9d7871ff64696b67065cf9d4863222a5
2021-10-07 23:29:41 -07:00
Tony Wickham
db0c2f1b79 Add flags to stash taskbar for app pinning and empty hotseat
Test: pin an app from overview, watch taskbar stash automatically until unpinning
Test: turn off suggestions for hotseat, remove all items and watch taskbar stash automatically when opening an app
Fixes: 190192993
Fixes: 193937948
Change-Id: Ia7260c60a820af1a48c9e4a400a52753baf34d41
2021-10-07 23:29:41 -07:00
Tony Wickham
772732de9e Migrate mStashedInApp to FLAGS_STASHED_IN_APP
This is to prepare for different flags that could cause taskbar to be stashed in an app without the user explicitly long pressing to stash.

Test: run wwdebug and wwlogcat, ensure still get logs for long press stash events; other interactions like clipping tasks to above unstashed taskbar still work as before
Bug: 190192993
Bug: 193937948
Change-Id: I90846e650a438e03bdcfdf9c4bf919e19cc5abb3
2021-10-07 23:29:34 -07:00
Alex Chau
0423e3b412 Merge "Only adjust min/max scroll for split screen in landscape" into sc-v2-dev am: fe69980f90 am: 5354bce2e1
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/15992132

Change-Id: I3901bbb4414a9cec2006ad7bb48d971358be04b8
2021-10-07 15:18:40 +00:00
Vinit Nayak
52025a6a52 Merge "Show split options when overview actions hidden for grid" into sc-v2-dev am: 7c2b52e5cf am: 34035ace74
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/15971858

Change-Id: I24e03b74abb4f64d8f94a3d114c82d68447a59a6
2021-10-07 15:18:29 +00:00
Vinit Nayak
38abfe1ea5 Merge "Move back to end of screen on keyguard" into sc-v2-dev am: c5ae15b481 am: 08b57cd012
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/15986998

Change-Id: Idb49286c7bdc5e27fbe09ab064020964e22face2
2021-10-07 15:18:15 +00:00
Alex Chau
fe69980f90 Merge "Only adjust min/max scroll for split screen in landscape" into sc-v2-dev 2021-10-07 14:52:05 +00:00
Vinit Nayak
7c2b52e5cf Merge "Show split options when overview actions hidden for grid" into sc-v2-dev 2021-10-07 14:51:44 +00:00
Vinit Nayak
c5ae15b481 Merge "Move back to end of screen on keyguard" into sc-v2-dev 2021-10-07 14:49:47 +00:00
Vinit Nayak
a08e806e25 Move back to end of screen on keyguard
* Animate it back into position after user unlocks
* Re-create taskbar on layout direction change so
PropertyHolders are recreated to take into account
new values

Fixes: 199852418
Test: Tested w/ password + pin for gesture + 3 button
Change-Id: Ie7f16f737a8fc12328c05d7628d0e3ae09fc08ca
2021-10-06 18:09:15 -07:00
Tony Wickham
c310a6f930 Merge "Fix issues with starting new recents animation before previous onRecentsAnimationStart" into sc-v2-dev am: f9d2082cf3 am: cdb2df88bd
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/15988588

Change-Id: I761d29cd74d833e259da29f422dd0bc9e144ff0c
2021-10-06 17:59:03 +00:00
Tony Wickham
f9d2082cf3 Merge "Fix issues with starting new recents animation before previous onRecentsAnimationStart" into sc-v2-dev 2021-10-06 17:36:45 +00:00
TreeHugger Robot
0e60eddf61 Merge "Remove workaround for unrotated gesture monitor (3/3)" 2021-10-06 17:18:05 +00:00
Alex Chau
aea42dc43e Only adjust min/max scroll for split screen in landscape
- Portrait's split placeholder is at top/bottom and does not affect scroll

Bug: 200537659
Test: Split left and split right in portrait / landscape, check if min/max scroll is correct
Change-Id: Ib7eb09d3cc44c8e0d962920ad094c73abc5f0dbd
2021-10-06 12:31:10 +01:00
Vinit Nayak
a85e762a62 Merge "Reset state after split launch animation finishes" into sc-v2-dev am: 6ab0d1e7be am: c8b7fef784
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/15986152

Change-Id: I8e71fbbb5344205209065a5c94a215db4020ba2a
2021-10-06 01:24:27 +00:00
Tony Wickham
66ed0ff23e Fix issues with starting new recents animation before previous onRecentsAnimationStart
- Fix logic for canceling animation for continued quick switch, so that this case (starting a new gesture before onRecentsAnimationStart() of the previous gesture) instead goes to the STATE_FINISH_WITH_NO_END flow.
- Update the end target so that we go to that state instead of always overview state if swipe was past the halfway threshold when we call endLauncherTransitionController(). This is specifically so we don't use OverviewInputConsumer on the second gesture, given the first one was canceled and didn't actually go to overview.
- GestureState#isRecentsAnimationRunning() now checks for STATE_RECENTS_ANIMATION_STARTED rather than _INITIALIZED, to be consistent with its javadoc and TaskAnimationManager#isRecentsAnimationRunning(). This also ensures we can correctly calculate continued quick switch (see above).
- Call cleanUpRecentsAnimation() before creating a new one in TaskAnimationManager. This ensures that the previous listener doesn't immediately cleanup the new gesture when it gets onRecentsAnimationCanceled() due to the new recents animation starting.

Test: swipe to home twice from the app, locally ignoring the onRecentsAnimationStart() from the first one, and ensure the second one responds normally
Bug: 193851085
Change-Id: I76e27c96b54293805546c0d6c82e77f975c69d7a
2021-10-05 18:14:51 -07:00
Vinit Nayak
6ab0d1e7be Merge "Reset state after split launch animation finishes" into sc-v2-dev 2021-10-06 00:56:48 +00:00
TreeHugger Robot
565c5ed6e2 Merge "Add TAPL test that dismissing focused task causes next task to become focused." into sc-v2-dev am: 1bf753f7a6 am: e77cae7f0d
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/15947750

Change-Id: I9c7cac5e73845ed4f08d2d518e661891188a0ce6
2021-10-06 00:45:44 +00:00
TreeHugger Robot
1bf753f7a6 Merge "Add TAPL test that dismissing focused task causes next task to become focused." into sc-v2-dev 2021-10-06 00:14:46 +00:00
Vinit Nayak
0516737146 Reset state after split launch animation finishes
Previously we were clearing SplitSelectStateController
state too soon for fake landscape case where
as soon as the tasks launch the device does a rotation
animation since the new split tasks are in landscape.
Move restting state after animation finishes.

Fixes: 201980666

Change-Id: I23efd59fad444c47ae6c2048888a8570f22610fd
2021-10-05 15:29:24 -07:00
Vinit Nayak
1772a30c70 Merge "Set SurfaceTransactionApplier on non-stale RemoteAnimationTargets" into sc-v2-dev am: 0ff2ad6ff4 am: 9b79688147
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/15956593

Change-Id: Ifb3b2498f5db96beadbb6edfe768d7b6762ab023
2021-10-05 22:03:37 +00:00
Vinit Nayak
0ff2ad6ff4 Merge "Set SurfaceTransactionApplier on non-stale RemoteAnimationTargets" into sc-v2-dev 2021-10-05 21:44:34 +00:00
Vinit Nayak
95ee358269 Show split options when overview actions hidden for grid
* We hide split option in task menu for focused task
if the user hasn't scrolled at all in overview
* We show option when user has scrolled because then
the split option from overview actions will be hidden
* Mostly uses the same logic as
RecentsView#updateActionsViewFocusedScroll()

Bug: 201380373
Test: Manual
Change-Id: Ia8769a67cfbb2e4414cd4d8e816e62245fc4bd2c
2021-10-05 14:28:07 -07:00
Alex Chau
b78753a73f Adjust min/max scroll according to mSplitPlaceholderSize am: 516584e648 am: 60818c3de0
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/15982847

Change-Id: I8cbf7c400b06c19b5dd37a1fff66007d414e9d61
2021-10-05 18:27:44 +00:00
Pat Manning
a140a0b611 Add TAPL test that dismissing focused task causes next task to become
focused.

Test: TaplTestsQuickstep.java
Bug: 197630182
Change-Id: I6f29dc6e3954f1b22ce8ffdf0a83db97202caefe
2021-10-05 18:00:11 +00:00
Vinit Nayak
01e47343e6 Set SurfaceTransactionApplier on non-stale RemoteAnimationTargets
* When we create RemoteTargetHandles we create a new
RemoteAnimationTarget from the old ones, which weren't
getting their addReleaseCheck() methods called
* The old recentsAnimationTargets were having it be set on them,
however they go unused.

Bug: 199043513
Change-Id: I36317f213139435ec09997290853787e30d11e0f
2021-10-05 10:40:12 -07:00
Alex Chau
516584e648 Adjust min/max scroll according to mSplitPlaceholderSize
- RTL is not handled yet due to split select broken

Bug: 200537659
Test: Split left and split right, check if min/max scroll is correct
Change-Id: I734d890bb955601da52cd806074014ed2e7eba55
2021-10-05 17:22:05 +01:00
Jerry Chang
93250cf918 Merge "Update side stage outline visibility with swipe gesture" into sc-v2-dev am: b001ce602f am: 6d6c9de783
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/15903128

Change-Id: I82bbca3b2d5bbecdaff5ba6627c87d30f69b227d
2021-10-05 01:19:23 +00:00
Jerry Chang
b001ce602f Merge "Update side stage outline visibility with swipe gesture" into sc-v2-dev 2021-10-05 00:46:36 +00:00
Prabir Pradhan
85f0cec3dc Remove workaround for unrotated gesture monitor (3/3)
There is currently a conflation between per-window-input-rotation and
unrotated gesture monitors. We would like to enable
per-widnow-input-rotation first without affecting gesture monitors so
that the old behavior is maintained for gesture monitors.

We can then add unrotated gesture monitors as a new feature later on.

Bug: 201982032
Bug: 179274888
Test: manual
Change-Id: I935165446e1a347129e9eb3cc816c62b24d4fc47
2021-10-04 06:06:18 -07:00
Jerry Chang
d41c032afd Update side stage outline visibility with swipe gesture
Bug: 200850654
Test: enabled side stage outline, observed the outline updates its
      visibility with swipe gesture properly.
Change-Id: I59fcb1fdbb07f5ad5327036972ebbe20ebeb11b7
2021-10-02 10:04:32 +08:00
TreeHugger Robot
889dfd07cd Merge "Don't snap to task when tapping task icon in overview grid" into sc-v2-dev am: f5942afeb5 am: a5a22a2970
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/15960194

Change-Id: I3b1826f21c6f643f87ddfa5022385e36005f9558
2021-10-01 19:46:43 +00:00
Alex Chau
d4864e5cbc Don't snap to task when tapping task icon in overview grid
- Open task menu directly instead of snapping to task, as there are spaces to do so unlike small screen

Fix: 201428316
Test: Tap task icon with clear all button shown in overview grid, no snapping
Test: Tap task icon with clear all button shown in non-grid overview, snapped to last task
Change-Id: Ief79f7c758c4109e5c7722b2f35647db98a9f05c
2021-10-01 18:23:25 +01:00
TreeHugger Robot
2e00cb8fee Merge "Overview - Remove Chips code" into sc-v2-dev am: d1ad84c6df am: 5587d7a493
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/15942463

Change-Id: I40a106e7ff4060c6c0459142defa418fad1d42b5
2021-09-30 22:43:30 +00:00
TreeHugger Robot
d1ad84c6df Merge "Overview - Remove Chips code" into sc-v2-dev 2021-09-30 22:14:41 +00:00
Alex Chau
9ec321cbe9 Switch to rest state when display changes am: 90d24f4b9b am: f017e1bfcc
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/15951262

Change-Id: If23c8dcd8a52c3293775a49b1e8e6b8efd336bc6
2021-09-30 19:40:29 +00:00
Alex Chau
90d24f4b9b Switch to rest state when display changes
Fix: 199463580
Test: Unfold device from overview (both from app and from home), overview should close
Change-Id: I7786f3ec46857075e123fa2a093c7b55ca616fc1
2021-09-30 17:26:05 +01:00
TreeHugger Robot
2e4a003f71 Merge "Remove the Share button option from Overview." into sc-v2-dev am: 70cc9ffe7c am: 8775a79741
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/15935390

Change-Id: I27bfb9c63fb618abb0faaa116e44b9e95aa84b45
2021-09-30 04:38:22 +00:00
TreeHugger Robot
70cc9ffe7c Merge "Remove the Share button option from Overview." into sc-v2-dev 2021-09-30 04:11:47 +00:00
Tony Wickham
ec7030a5e9 Merge "Aniamte stashed handle color changes" into sc-v2-dev am: 6db0cf1e22 am: 631a02507d
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/15943405

Change-Id: I541418eadc98644d9b6f99a7a5a30dea519b1386
2021-09-30 02:39:20 +00:00
Tony Wickham
f6af104676 Merge "Update stashed handle region to only include the handle" into sc-v2-dev am: ad3de8eab8 am: b3ade306da
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/15943399

Change-Id: I06b1964118f83bc74678c8ea7a00341c24cf17dd
2021-09-30 02:39:10 +00:00
Tony Wickham
6db0cf1e22 Merge "Aniamte stashed handle color changes" into sc-v2-dev 2021-09-30 02:17:18 +00:00
Tony Wickham
ad3de8eab8 Merge "Update stashed handle region to only include the handle" into sc-v2-dev 2021-09-30 02:17:03 +00:00