summaryrefslogtreecommitdiffstats
path: root/src/com/android/launcher3/allapps
Commit message (Collapse)AuthorAgeFilesLines
* Scaling down hotseat to match prediction row size when going to overviewSunny Goyal2019-11-061-11/+0
| | | | | Bug: 141265005 Change-Id: I3b11146881af334508f553e4ca3a36b9291511d9
* Support defining X column layout for all apps as a display option.Jon Miranda2019-09-162-2/+7
| | | | | Bug: 124967099 Change-Id: I7bf576759b3fa4f6ca617fbbd660541c12fd09ac
* Merge "Properly prevent All Apps relayouts by avoiding scrollToPosition" ↵Tony Wickham2019-09-092-10/+9
|\ | | | | | | | | | | | | | | into ub-launcher3-qt-qpr1-dev am: d785fc289c Change-Id: I69738f5d9c1f77195f2f82f963958d3cc741e9af
| * Merge "Properly prevent All Apps relayouts by avoiding scrollToPosition" ↵Tony Wickham2019-09-052-10/+9
| |\ | | | | | | | | | into ub-launcher3-qt-qpr1-dev
| | * Properly prevent All Apps relayouts by avoiding scrollToPositionTony Wickham2019-09-032-10/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Calling scrollToPosition on RecyclerView internally calls requestLayout() (to cacluate where to scroll and then go there). Therefore, we should avoid calling that whenever possible, especially during transitions. In particular, we can optimize scrollToTop() to not scrollToPosition() if we are already at the top. This makes some other workarounds unnecessary, namely setting All Apps to GONE during system gestures. Test: Open an app, swipe up, ensure AllAppsRecyclerView doesn't get onLayout(). If we had scrolled to an app first, we get one layout in prepareRecentsUi(), but not during the transition. Bug: 140308849 Change-Id: I62ee341bf5893c121cfc013cc6542559f79d2a42
* | | Merge "Removing flinging via UI Automator" into ub-launcher3-qt-qpr1-devvadimt2019-09-091-9/+0
|\| | | | | | | | | | | | | | | | | am: e1335c2359 Change-Id: Iad5c6c0bf74ac223cc60b64fa7a15dae544d500f
| * | Removing flinging via UI Automatorvadimt2019-09-041-9/+0
| |/ | | | | | | | | | | | | UI Automator loses events. Switching to the homemade scrolling. Bug: 138729157 Change-Id: Ib09bd5714e83eadfc65b54202867bf375e6b47c5
* | Don't update all apps content visibility during home animationTony Wickham2019-08-302-6/+12
|\| | | | | | | | | | | am: 7310bb4b5c Change-Id: I92382497e4444e2ca1d1951c44646bc5047a6f36
| * Don't update all apps content visibility during home animationTony Wickham2019-08-302-6/+12
| | | | | | | | | | | | | | | | | | We already set the all apps content visibility = GONE at the start of the gesture to prevent relayouts, but when animating home we were inadvertently changing it to INVISIBLE, causing a relayout and jank. Bug: 140308849 Change-Id: I285746f8ac8f3f857282e22ebec8eebd0b98647f
* | Merge "Mouse wheel scroll apps with work profile enabled instead of ↵TreeHugger Robot2019-08-231-0/+5
|\ \ | | | | | | | | | switching tabs" into ub-launcher3-master
| * | Mouse wheel scroll apps with work profile enabled instead of switching tabsSamuel Fufa2019-08-231-0/+5
| | | | | | | | | | | | | | | | | | Test: Manual Bug:79164903 Change-Id: I4b382034ea5b91b24dcc158bd9fa46d64f54cc7a
* | | Merge "Not freezing All Apps during user interactions" into ↵Vadim Tryshev2019-08-222-19/+1
|\ \ \ | |/ / |/| / | |/ | | | | | | | | ub-launcher3-qt-qpr1-dev am: 94dcb3b5bf Change-Id: I110730f8abb1d63c363aba9939799570bea69f07
| * Not freezing All Apps during user interactionsvadimt2019-08-222-19/+1
| | | | | | | | | | | | | | | | | | | | | | | | Extremely rarely, All Apps will get stuck in a non-updating state even after the interaction ends. It would be impractical to try drilling to the root cause of this, so it's better to just allow All Apps updates while the user interacts with it. Bug: 139384936 Change-Id: I2ed7fb052da77a9e47ef9b9aa7800499071b98c3
* | Merge "Moving section names to model, to avoid an unnecessary hashmap" into ↵TreeHugger Robot2019-08-131-28/+2
|\ \ | | | | | | | | | ub-launcher3-master
| * | Moving section names to model, to avoid an unnecessary hashmapSunny Goyal2019-08-061-28/+2
| | | | | | | | | | | | Change-Id: I9620367e0c2203b2290a4a188411ccb9174ae130
* | | Merge "Fix DEFER_UPDATES_NEXT_DRAW flag is not unset when binging happens ↵Sunny Goyal2019-08-051-0/+4
|\ \ \ | |/ / |/| / | |/ | | | | | | | | twice" into ub-launcher3-qt-r1-dev am: 0a9e91b6e4 am: 2c848d029c Change-Id: I827430185fbab9433f74f402f382cb004ab744c3
| * Fix DEFER_UPDATES_NEXT_DRAW flag is not unset when binging happens twiceSunny Goyal2019-08-051-0/+4
| | | | | | | | | | Bug: 138646754 Change-Id: I76eccdb812013299e18f197ba17ff858a01a6338
* | Merge changes Ib3e373c2,Ia5d6f545 into ub-launcher3-qt-r1-dev am: 1343f03e2eTony Wickham2019-07-314-19/+21
|\| | | | | | | | | | | am: 70f539cd45 Change-Id: Ifddcba1cd63950424f88ad1d8e01fa9a11f73a2e
| * Tie prediction text to all apps interpolatorTony Wickham2019-07-314-19/+21
| | | | | | | | | | | | | | | | | | | | | | | | Now floating headers get 2 interpolators: one for the header content itselt, and one for the all apps content that follows. That way, they can choose to intperolate part of their content as if it were part of all apps instead of the header. Currently, we do this to animate predicted icons quickly, followed by the all apps icons, predictions text, all apps scrollbar, and all apps divider as you continue swiping. Bug: 132455160 Change-Id: Ib3e373c291e174e1306a53854d0ad4dc29eb4b76
* | Separating apps list callback from model callback and simplifying various eventsSunny Goyal2019-07-251-30/+17
|/ | | | | Bug: 137568159 Change-Id: I4d8bb5c01cabe642731ebd3f57fe15bdc5a569b7
* Merge "Fill the logging container in the AllAppsContainerView" into ↵Becky Qiu2019-07-121-1/+6
|\ | | | | | | ub-launcher3-qt-r1-dev
| * Fill the logging container in the AllAppsContainerViewBecky2019-07-111-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This would fix the issue that when drag and drop an icon from the all apps page, it doesn't have the right container information logged. Logging after this fix: 07-11 10:57:04.392 12969 12969 D UserEvent: action:DRAGDROP 07-11 10:57:04.392 12969 12969 D UserEvent: Source child:APP_ICON parent:ALLAPPS 07-11 10:57:04.392 12969 12969 D UserEvent: Destination child:APP_ICON parent:WORKSPACE id=0 07-11 10:57:04.392 12969 12969 D UserEvent: Elapsed container 744 ms, session 17440 ms, action 739 ms Test: manual Bug: 111935715 Change-Id: Ifb078e57697b051e3a527c16abaad40663eae687
* | Remove tracing for a fixed bugvadimt2019-07-112-8/+0
| | | | | | | | | | Bug: 132900132 Change-Id: Ic4ce3669f88c932e392b7517ec9fda11384dc334
* | Removing tracing for a fixed bug.vadimt2019-07-111-4/+0
|/ | | | | Bug: 133867119 Change-Id: I796118f5ff0c27db002bb0e3369e651c95b06bbe
* Cleaning up some animation states:Sunny Goyal2019-06-211-4/+8
| | | | | | | | | | | | > When running one-off animations during quickstep, cancelling prevoisly running animations. > Cancelling such one-off animations when state is reset > Preventing touch proxied from recent transition to affect quickswitch (by affecting pagedView) Bug: 135686388 Bug: 135571566 Change-Id: Id647015a583761d8fd46a02e3e2d88027e282a79
* Using FloatProperty for spring animation, instead of a interfaceSunny Goyal2019-06-191-10/+6
| | | | | | to allow easier generalization of animation definitions Change-Id: I37b1a604003ec007aa390eabdfe8c1ab733b7471
* Fix bugs with hotseat in overviewTony2019-06-181-4/+12
| | | | | | | | | - Allow touches to go through recents to the hotseat. - Translate the hotseat with the all apps shelf when swiping up in background app state. Bug: 135222111 Change-Id: Ib887fc25ccfeb406a44074198c11f7b1d245443c
* Merge "More tracing for app not starting from all apps" into ub-launcher3-qt-devTreeHugger Robot2019-06-131-0/+4
|\
| * More tracing for app not starting from all appsvadimt2019-06-131-0/+4
| | | | | | | | | | | | Bug: 132900132 Bug: 134172603 Change-Id: I6a85e616552e84b8165661d25ed4330f55d3946f
* | Optimizing some layouts in taskviewSunny Goyal2019-06-131-8/+3
| | | | | | | | | | | | | | | | | | > Recycling DigitalWellBeingToast so that the view is not inflated everytime > Simplifying DigitalWellBeingToast to use a single text view > Adding support for footers in taskView without creating additional layout Bug: 122345781 Change-Id: Ia889819b93eb8644532ea95c6767554874d5e2d1
* | Freezing all apps updates during certain testsvadimt2019-06-062-12/+24
|/ | | | | | | | | | | | | | | | | | | | | | This CL adds a very low risk because most (but not all) changes affect only Launcher behavior during the test. This should fix a lab-only flake when all apps keeps changing while the test is working with it. Example: test figures out which icon to click, by the moment it clicks there, there is another icon there, or the icon is under the search box, and clicking opens IME. Switching test devices to airplane mode didn't help. The earlier change that prevents popup menu cancellation is not general enough. Now the tests are given an API to explicitly freeze and unfreeze all-apps, which should be a final solution. Bug: 132900132 Bug: 133765434 Change-Id: I8b81cc9be004482beb6cdcdd05406e2d9b4c7629
* Merge "Fixing All Apps's getting stuck in a non-updating state" into ↵TreeHugger Robot2019-06-041-2/+3
|\ | | | | | | ub-launcher3-qt-dev
| * Fixing All Apps's getting stuck in a non-updating statevadimt2019-06-031-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Normally, if all apps is open, and an app installs/uninstalls/updates, All Apps will immediately reflect this. However, depending on something subtle in the intensity of the swipe gesture that brought All Apps, all apps will freeze, and won't update. This frozen state will go away after scrolling in all apps, iteration with an icon and, generally, any tap interaction with All Apps. Otherwise, it will stay until it's closed and opened again via a gesture with a different pattern. The reason is in the code that freezes All Apps updates during user interactions with all apps. For example, during scrolls. Or while the user holds an icon, expecting to see a context menu; in this case an update would cause the menu to not appear, which is somewhat annoying. The motivation to add this code was to solve a category of lab-only flakes when a context menu couldn't open because the lab device was busy with post-flash activities. The code incorrectly assumed that after ACTION_DOWN, we are guaranteed to get either UP or CANCEL, which is wrong. It's after *consumed* ACTION_DOWN that we'll get these events. The fix still solves the user's and tests' problems that the code was supposed to solve. Bug: 134442147 Change-Id: I9db74a33ecf93b1dc6bc69df301f7f542dea2a40
* | More debug tracing for switching to all apps not sending the final eventvadimt2019-05-311-0/+6
|/ | | | | | | | (to the test) Bug: 134172603 Bug: 133867119 Change-Id: I00cf3a35daefa23d2e72a89d21e6ce37c8a9a46d
* Merge "More logging for lab-only flake when an app doesn't start" into ↵TreeHugger Robot2019-05-291-0/+6
|\ | | | | | | ub-launcher3-qt-dev
| * More logging for lab-only flake when an app doesn't startvadimt2019-05-281-0/+6
| | | | | | | | | | | | | | I suspect that All Apps gets refreshed around the time of the click. Bug: 132900132 Change-Id: I6d3053f7d8f998e08ca495c005e14ddece634164
* | Fix bug where DiscoveryBounce is not cancelled after state change.Jon Miranda2019-05-281-0/+15
|/ | | | | Bug: 131768436 Change-Id: I7d389671969a75ae7ec9f9f5cc1326f7105e2071
* Launcher reports whe 0-button swipe-up gesture pause is detected.vadimt2019-05-241-1/+1
| | | | | | | | | This eliminates an unreliable timeout. Also removing an unnecessary check for harness that is done by the called method. Change-Id: If954580060415cbb2952532c16ea0ae4dc7b9469
* Merge "Remove unnecessary dispatchInsets routing" into ub-launcher3-qt-devTreeHugger Robot2019-05-241-1/+11
|\
| * Remove unnecessary dispatchInsets routingSunny Goyal2019-05-211-1/+11
| | | | | | | | | | | | | | | | Insets are dispatched to all views. We never consume the insets and let each view decide what to do with it. Also fixing scrim in all-apps when in full-gesture navigation Change-Id: Ib1c6bef5b32aac0c6ea03078b5138d2d0408c6d8
* | Moving TestInformationProvider to Launcher3 so that it can be used forSunny Goyal2019-05-231-1/+1
| | | | | | | | | | | | | | | | testing Launcher3 without quickstep Also keeping the provider as disabled until needed Change-Id: Ib5f459e02ae551724b390f3b74f43d601568d749
* | Fix issue where AllAppsContainerView was blocking touches to part of the ↵Adam Cohen2019-05-221-0/+8
|/ | | | | | | | | | | | | | home screen => After the scrubber was engaged, it would continue to intercept touches forever (seems like a long standing issue) => Also fixed issue where the All Apps scrubber could be engaged from the Home / Overview states b/133265591 b/132716177 Change-Id: I8c7b9d45be65216f2f1a69f69ab1636accd812c0
* Using model-time scrolling in all appsvadimt2019-05-142-4/+14
| | | | | | | This should solve flakes when the test thread wakes up too rarely to inject enough touch events. Change-Id: I461583d35eb4bfe0192b81c242aacf8d20e353d1
* Disabling updating All Apps during a long-press recognitionvadimt2019-05-091-0/+14
| | | | | | | | | | | | | | | | | | | This is supposed to eliminate test flakes in the lab. When an app gets updated while all apps is opened, all-apps will be re-laid-out. If the layout happens while we are recognizing a long-press on an app icon in all Apps, the long-press won't be recognized. In the lab, this happened so frequently that it caused considerable flakiness. Even after we started running tests in airplane mode, this still keeps happening. The bug refers an example with airplane mode on. The fix will be also mildly beneficial for users. Bug: 132177321 Change-Id: Ie7c7473fe94b8af83f04cd719286bae69e2d9de0
* Makes Launcher hide the all apps content and scrim on assistant visible.Govinda Wasserman2019-04-232-1/+15
| | | | | | | | | | | This hides the quick search bar and the suggested apps. Visibility is unchanged when the all apps view is fully expanded. Test: Tested locally Change-Id: I6bc453b5ad89ca3d1280957e595bf2a3f074a72d BUG:129755311 FIX:129755311 (cherry picked from commit 917af755e42a03cdc43ed1c5f897f585f2041725)
* Add QuickSwitchTouchController on homeTony2019-03-281-1/+1
| | | | | | | | | - Added QuickSwitchState, which we animate to when swiping right on the nav bar from NORMAL state - Task launches when the state transition to QuickSwitchState ends Bug: 126596417 Change-Id: Id66650401d817703fc6d044fb26a25cccbc07e11
* Adding a utility class to listen for app launchesSunny Goyal2019-03-262-7/+6
| | | | Change-Id: I62e82a6e04b7101773d98c6e7aec574facd053fe
* Quickly fade in QSB when swiping up when SWIPE_HOME = falseTony Wickham2019-03-111-1/+4
| | | | | | | | | Now that the "hook" gesture is supported for old quickstep to switch apps, we need to fade out the QSB so it doesn't overlap the TaskView we are switching to. Bug: 111926330 Change-Id: Ie04c7909f6b96d12c810b438db89eedf8ea5bdeb
* Scale and translate hotseat with workspace instead of all appsTony2019-03-031-6/+0
| | | | | | | | | - Move the hotseat alongside workspace instead of on top of all apps in xml layout - Set pivot point of hotseat to match the workspace's, and apply the same scale - Translate the hotseat with the workspace instead of all apps - SpringLoadedState does not scale or translate the hotseat Change-Id: Ic45fe99f83f0e0012afa78073d9577e65da444e2
* Moving some utility methods aroundSunny Goyal2019-01-311-2/+3
| | | | Change-Id: I8abca49a0dbf656212b21e0552502036a1619164