Commit Graph

13917 Commits

Author SHA1 Message Date
TreeHugger Robot c6d0a6ca82 Merge "Do not add HOTSEAT_ICONS to visible elements for device type "phone" in ALL_APPS State" into tm-dev am: 002e1fcde5 am: 5b8be11c37 am: 8fdcfe0520 am: 65e2d9dbb3
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/18084536

Change-Id: Ia5bdb75b36390dbae8fc9baf2a8d7d595ea1d9f3
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-05-02 12:55:29 +00:00
TreeHugger Robot 518d03090a Merge "Removing ununsed append logic in search" into tm-dev am: 4adce848da am: 509805b131 am: 1c302cb140 am: 1e78a8d9cb
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/18091362

Change-Id: Ia5a350c2f7e2b62f57b3e8c49030a605840b2ade
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-05-02 11:03:44 +00:00
Thales Lima 1b05596110 Fix hotseat width to span a number of columns am: 828875cd12 am: ab5faf06a8 am: 14527d5f6d am: 313df994c6
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/17983445

Change-Id: I31876f36399c28dc810194623a8d44f817e1853a
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-05-02 11:01:06 +00:00
TreeHugger Robot 002e1fcde5 Merge "Do not add HOTSEAT_ICONS to visible elements for device type "phone" in ALL_APPS State" into tm-dev 2022-04-30 17:12:35 +00:00
Anushree Ganjam f6254c5947 Do not add HOTSEAT_ICONS to visible elements for device type "phone" in
ALL_APPS State

HOTSEAT_ICONS got added to visible elements for ALL_APPS State in ag/16992837.

============================================================
Phone
Before:
HotSeat present in view hierarchy - https://hsv.googleplex.com/4906040983289856?node=41
Video: https://b.corp.google.com/issues/228803923#comment9

After:
HotSeat not present in view hierarchy - https://hsv.googleplex.com/5379653922455552?node=42
Video: https://b.corp.google.com/issues/228803923#comment11
============================================================

Tablets

I couldn't test this implementation on tablet. But I have verified on a phone
locally by adding HOTSEAT_ICONS to visible elements and then verifying
"IMPORTANT_FOR_ACCESSIBILITY_NO_HIDE_DESCENDANTS" get set and hotseat icons are not focusable in all apps.
Video : https://b.corp.google.com/issues/228803923#comment14

============================================================

Bug: 228803923
Bug: 227565451

Test: Manual. See the video attached in bug.
Change-Id: I055e3bb61c8cfd240e0fe1000fe3a391a150f6d1
2022-04-29 23:33:48 +00:00
TreeHugger Robot 4adce848da Merge "Removing ununsed append logic in search" into tm-dev 2022-04-29 22:55:13 +00:00
Schneider Victor-tulias c4de960bbf Merge changes from topic "taskbar-misc-fixes-master"
* changes:
  Translate the 3 navigation buttons to/from their in-app position when animating to/from the -1 screen, all apps and widgets.
  Fix taskbar 3-button truncation on rotate
2022-04-29 18:17:14 +00:00
Sunny Goyal 3091f0a4fe Removing ununsed append logic in search
Bug: 229860311
Test: Manual
Change-Id: Id6a72784b186fc5f0aaeac52f2a66fa35711fe4c
2022-04-29 09:49:50 -07:00
Thales Lima 828875cd12 Fix hotseat width to span a number of columns
Bug: 227344807
Test: manual
Change-Id: Ice4e37929bcbb8c9cf3af340e67b6bec002578bd
2022-04-29 14:56:30 +00:00
TreeHugger Robot 0d22a0b73e Merge "Revert "Remove old logs associated to b/200572078"" 2022-04-28 20:53:35 +00:00
Raj Garg 48f8a78edb Merge "Making the parseAndAddDeepShortcut api private." into tm-dev am: 0191a02a02 am: d9a9e76805
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/17988066

Change-Id: I427223faaa109e9f2ebfb5bb3987db0d3d5be68c
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-04-28 11:15:47 +00:00
Raj Garg 0191a02a02 Merge "Making the parseAndAddDeepShortcut api private." into tm-dev 2022-04-28 09:58:37 +00:00
Anushree Ganjam e9466b3f94 Merge "Revert "Refactors Search results into separate RV for Toast."" into tm-dev am: fa693be19f am: 168362edd2
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/18011182

Change-Id: I62f302fbdaf1b37988f6a56c8cf501a454cb2382
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-04-28 06:03:36 +00:00
Raj Garg 0b3d54a4f6 Merge "Adding support to parse deep-shortcuts." into tm-dev am: f3924d62b0 am: d611a22993
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/17956680

Change-Id: Ie5f318cbebef7b60011ea0fe17a9d02a3cab5918
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-04-28 06:03:19 +00:00
My Name 9a6dbac68c Making the parseAndAddDeepShortcut api private.
There is no need to expose this api. Missed the comment https://googleplex-android-review.git.corp.google.com/c/platform/packages/apps/Launcher3/+/17956680/comments/1c9b7443_cc54296d

Test: Tested manually on device.
Change-Id: If71e203f32a59fa59c406303411a37cba7b6c62e
2022-04-28 05:17:40 +00:00
Anushree Ganjam fa693be19f Merge "Revert "Refactors Search results into separate RV for Toast."" into tm-dev 2022-04-28 04:52:01 +00:00
Raj Garg f3924d62b0 Merge "Adding support to parse deep-shortcuts." into tm-dev 2022-04-28 04:32:38 +00:00
Anushree Ganjam 5cc7ff0990 Revert "Refactors Search results into separate RV for Toast."
This reverts commit 6729f0b950.

Reason for revert: This change caused b/230648542.

Please see https://b.corp.google.com/issues/230648542#comment5 for the video after reverting this change.

Bug: 206905515
Bug: 230648542

Change-Id: I85f063c56cad137c05b810204244bba7e8f94ee7
2022-04-28 01:37:49 +00:00
Sebastián Franco 880c3d6648 Merge "Give the tests the ability to emulate other devices screens" into tm-dev am: 84051cd8bf am: 3ee4b74621
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/17413625

Change-Id: I2e8a9b73dd7225e01dfc0a4dd4498943c7090e97
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-04-27 19:06:20 +00:00
TreeHugger Robot b6640e6402 Merge "Move spring loaded scale calculation to DeviceProfile." into tm-dev am: 84bff57d8d am: 9a400618db
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/17909525

Change-Id: I0df2f35011649cf30e42ac741fbea01bcc23ee91
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-04-27 19:06:04 +00:00
András Klöczl b2372e19ca Merge "Change home dragging logic for dead zones" into tm-dev am: 143b71203f am: ecfbf14f2b
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/17217228

Change-Id: Ia810a31852dcc06d7e36343192446bc9d3ca234b
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-04-27 19:05:50 +00:00
Alex Chau a95d18b430 Merge "Update overview actions alignment" into tm-dev am: a80efcee9f am: 0a4e0923d2
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/17888303

Change-Id: Ief8f521e87f98491cd777e02cf90461a8409b9a0
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-04-27 19:05:10 +00:00
Sebastián Franco 84051cd8bf Merge "Give the tests the ability to emulate other devices screens" into tm-dev 2022-04-27 17:25:55 +00:00
Sunny Goyal a0b7c53e50 Revert^2 "Removing some unused code from All-Apps"
13ae6784b5

Change-Id: I06a4c9cc6beba4fc87b4a9375872c68e71e1c5f6
(cherry picked from commit 52f4c16a94)
2022-04-27 13:53:03 +00:00
TreeHugger Robot 84bff57d8d Merge "Move spring loaded scale calculation to DeviceProfile." into tm-dev 2022-04-27 13:51:59 +00:00
András Klöczl 143b71203f Merge "Change home dragging logic for dead zones" into tm-dev 2022-04-27 12:44:34 +00:00
Alex Chau a80efcee9f Merge "Update overview actions alignment" into tm-dev 2022-04-27 12:24:24 +00:00
My Name 2f5a1b6756 Adding support to parse deep-shortcuts.
Actual problem space is to prebundle YouTube Shorts shortcut to the
JioPhone Next device. The parsers don't support parsing the deep shortcut. In this CL we have added support for parsing deep-shortcuts.

Test: Tested manually on device.
Bug: 227290255
Change-Id: I7ea240a29639b88254fa679733541ff98ddfaa48
2022-04-27 08:14:46 +00:00
Sunny Goyal 52f4c16a94 Revert^2 "Removing some unused code from All-Apps"
13ae6784b5

Change-Id: I06a4c9cc6beba4fc87b4a9375872c68e71e1c5f6
2022-04-27 03:22:58 +00:00
Tracy Zhou 4622349c69 Merge "Add originalView of the menu to SystemShortcut" into tm-dev am: 7183e4b460 am: dbb368b40f
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/17677218

Change-Id: I7d029e2545795697f30dcb064139d48cc525b6ba
Ignore-AOSP-First: this is an automerge
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-04-26 18:24:49 +00:00
Tracy Zhou 7183e4b460 Merge "Add originalView of the menu to SystemShortcut" into tm-dev 2022-04-26 17:28:40 +00:00
TreeHugger Robot cea10a4fb4 Merge "Refactors Search results into separate RV for Toast." into tm-dev am: 0a6e9b4751 am: eb4fa04985
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/17699868

Change-Id: I76d64e3b7cd443423b3bbe291e82717c4f4d2ffa
Ignore-AOSP-First: this is an automerge
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-04-26 16:11:14 +00:00
Andras Kloczl 0056e95d8e Change home dragging logic for dead zones
- If a dragged item is not over a valid drop target
then we don't highlight anything and on drop we move
it back to its original position.
- Instead of checking if the dragged item is over the
hotseat view, now we check if the item is over the hotseat
shortcuts.
- Smartspace is a dead zone

Test: drag items from different workspaces, allapps, widget
  picker etc. to all parts of the screen and verify which
  view is targeted and where the item gets dropped.
  Check on phone/foldable/tablet.
Bug: 228201267
Change-Id: I18de576ea300d58ae645e511b238e0b946b5595a
2022-04-26 16:28:11 +01:00
Schneider Victor-tulias a9b2c4bb5a Revert "Remove old logs associated to b/200572078"
This reverts commit c4240fd429.

Reason for revert: Returning logs for flaking test

Bug: 227650070
Change-Id: I7db31533cfb72d9e329f38dc001f3a8e52b9fb7f
2022-04-25 22:23:20 +00:00
Schneider Victor-tulias 9c1a452a1d Translate the 3 navigation buttons to/from their in-app position when animating to/from the -1 screen, all apps and widgets.
Bug: 221455508
Test: opened all apps, widgets, -1 screen, notifications shade and keyboard in various combinations and orders; locked screen, launched app, returned home with the back/home buttons, opened overview
Change-Id: Ia0b406aacf72b34bd6b7ff1c01278ab6895a7da4
2022-04-25 14:57:13 -07:00
Alex Chau a2fc764405 Update overview actions alignment
- Align overview actions to the top with a defined margin below task thumbnail, except for vertical bar layout
- In modal state, use task view's bottom as pivot, and expand using all the space about pivot
- Simplified overview actions margin calculation and moved some logic to DeviceProfile

Bug: 209981696
Test: manual on various devices
Change-Id: I4c96cb02c6f3d6adf30633ddeb883bc4eec16f36
2022-04-25 19:20:21 +01:00
Pat Manning a2e1499a2f Move spring loaded scale calculation to DeviceProfile.
Fix: 229838395
Test: manual
Change-Id: I6dbbc850e88aaacceb1363e342404b06104f8c10
2022-04-25 18:58:09 +01:00
Sebastian Franco 00aff95ac0 Give the tests the ability to emulate other devices screens
This code contains utility clases that can change the display
of a device and make it look like another device.
The function DisplayEmulator#emulate receives a DeviceEmulationData
a certain grid to emulate and a callback, everyting that happens
inside of the callback will happen when the device is being emulated
and can be used by other tests.

Example test:
package com.android.launcher3.deviceemulator;
import androidx.test.ext.junit.runners.AndroidJUnit4;
import androidx.test.filters.MediumTest;
import com.android.launcher3.deviceemulator.models.DeviceEmulationData;
import com.android.launcher3.ui.AbstractLauncherUiTest;
import org.junit.Test;
import org.junit.runner.RunWith;
import java.util.concurrent.TimeUnit;

@MediumTest
@RunWith(AndroidJUnit4.class)
public class TestTest extends AbstractLauncherUiTest {
    @Test
    public void testEmulation() throws Exception {
        String deviceCode = "pixel6pro";
        DeviceEmulationData deviceData = DeviceEmulationData.getDevice(deviceCode);
        String grid = "normal";
        DisplayEmulator displayEmulator = new DisplayEmulator(mTargetContext);

        displayEmulator.emulate(deviceData, grid, () ->{
            TimeUnit.SECONDS.sleep(10);
            return true;
        });
    }
}

Test: You could use the test above to make your device look like a
Pixel6 pro for 10 secons.
Fix: 229028257

Change-Id: Icd79be405a2e14dda0bc5f555b0e46149e16f912
2022-04-25 11:57:58 -05:00
Andy Wickham d40fd3a42b Merge "Adds Search Onboarding count and renames existing edu pref." into tm-dev am: 833b0a0b5b am: 6ca6258750
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/17876281

Change-Id: Ic57445a0a9415a8e3e8c96c226065b946f670344
Ignore-AOSP-First: this is an automerge
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-04-25 01:13:39 +00:00
Andy Wickham 6729f0b950 Refactors Search results into separate RV for Toast.
This will help enable transitions between A-Z apps lists and
search results because both can be seen simultaneously and
manipulated independently.

Some high level items of the refactor:
 - SearchRecyclerView is added; logic that populated the main
   (personal) tab with search results was simply redirected to
   this RV instead.
 - BaseAllAppsContainerView added isSearching() method. Returns
   false, and ActivityAllAppsContainerView overrides (as search
   is handled there).
 - Renamed BaseRecyclerView to FastScrollRecyclerView to better
   describe what it does. SearchRecyclerView extends this, but
   returns false for supportsFastScrolling().
 - AlphabeticalAppsList#mAllAppsStore is now optional, so the
   Search RV doesn't need to store/listen to apps. Note this
   doesn't affect the predicted app row which is still updated
   if one of the predicted apps is uninstalled (I tested this).

Future work:
 - Determine why dispatchRestoreInstanceState is not called for
   BaseAllAppsContainerView. Save is called, e.g. on rotation.
   Effect of restore not called: rotating while searching goes
   back to A-Z list.
 - Keep suggested apps in Header while searching. Currently they
   are rendered in the SearchRV above search results, as before.
 - Potentially extract Personal/Work tabs to move independently of
   header.
 - AlphabeticalAppsList is a misleading name because it can also
   contains search results. However, things are pretty intertwined
   between that and BaseAllAppsAdapter (effectively a circular
   dependency), so I figured cleaning all that up was out of the
   immediate scope of this refactor, which is mainly meant to
   unblock transition work.

Bug: 206905515
Test: Manually checked for regressions, ran tests.
Change-Id: I4d3757c8a8f9b774956ca6be541dd4fcdad1de13
2022-04-24 17:36:48 -07:00
Andy Wickham 833b0a0b5b Merge "Adds Search Onboarding count and renames existing edu pref." into tm-dev 2022-04-25 00:26:51 +00:00
Tony Wickham efdae030a6 Merge "Address a couple small followups for TaskbarInsetsController" into tm-dev am: 357c622572 am: b8bfa20d79
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/17873349

Change-Id: I14e26af5f266999bbc7f7a3ad528f96fa652ebff
Ignore-AOSP-First: this is an automerge
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-04-22 23:42:28 +00:00
Tony Wickham 357c622572 Merge "Address a couple small followups for TaskbarInsetsController" into tm-dev 2022-04-22 22:55:10 +00:00
Sunny Goyal ba0a00eec9 Merge "Removing fade in logic during load." into tm-dev am: 84151d2bca am: 77d7797f83
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/17768817

Change-Id: Icaa4a4a6863e43f6b499d0c21be0614e11b4381c
Ignore-AOSP-First: this is an automerge
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-04-22 21:19:48 +00:00
Sunny Goyal 84151d2bca Merge "Removing fade in logic during load." into tm-dev 2022-04-22 20:29:34 +00:00
Sunny Goyal 62e3036a27 Merge "Revert "Removing some unused code from All-Apps"" into tm-dev am: b1f6a3fe97 am: cadec37267
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/17917348

Change-Id: Ib3c7540e82a1e148c87f773890d6d3d66c371a3b
Ignore-AOSP-First: this is an automerge
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-04-22 20:22:21 +00:00
Sunny Goyal a5536d70fe Merge "Removing some unused code from All-Apps" into tm-dev am: a938cc27a2 am: 59f7c407f1
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/17867143

Change-Id: Iee09d756f2c367faf63db07a6647f5fae7fc835f
Ignore-AOSP-First: this is an automerge
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-04-22 20:19:24 +00:00
Sunny Goyal b1f6a3fe97 Merge "Revert "Removing some unused code from All-Apps"" into tm-dev 2022-04-22 19:55:42 +00:00
Sunny Goyal 13ae6784b5 Revert "Removing some unused code from All-Apps"
Revert submission 17866780-cancel-request

Reason for revert: wrong icons

Reverted Changes:
Iedc98e5ef:Cancelling previous request and clearing UI when a...
I72394fb42:Removing some unused code from All-Apps

Change-Id: I2ecefcaf07e6105a10f86b8dc19f75089e45a3e0
2022-04-22 19:53:37 +00:00
Sunny Goyal a938cc27a2 Merge "Removing some unused code from All-Apps" into tm-dev 2022-04-22 19:40:06 +00:00