Commit Graph

13866 Commits

Author SHA1 Message Date
Andy Wickham
34e8c3364a Merge "Fixes Search bar padding when work profile not present." into tm-dev 2022-06-10 17:07:43 +00:00
Sunny Goyal
ad7e199e52 Merge "Clone the drawable lazily instead of on the UI thread while initializing" into tm-dev 2022-06-10 15:28:30 +00:00
Becky Qiu
f13d3cca63 Merge "[Toast] Add a new latency type as FAIL." into tm-dev 2022-06-09 22:28:09 +00:00
Andy Wickham
918ba0c9f2 Updates keyboard option feature flag name.
This should ignore any value previously stored for the old flag.

Fix: 235440407
Test: Manual
Change-Id: Ib0b21442443575e88153f5a13ea945bfad3fa9f0
2022-06-09 11:02:39 -07:00
Andy Wickham
e44b026880 Merge "Fixes issue where personal/work tabs could overlap search reasults." into tm-dev 2022-06-09 17:47:54 +00:00
Alex Chau
1965e1309c Merge "Use DISPLAY_CATEGORY_ALL_INCLUDING_DISABLED for getDisplays" into tm-dev 2022-06-09 17:40:07 +00:00
Sunny Goyal
0d88d0088a Clone the drawable lazily instead of on the UI thread while initializing
Bug: 234764579
Test: Manually
Change-Id: I557a532e4f71475e9294854d17068007a2eb7338
2022-06-09 09:59:42 -07:00
Alex Chau
2545e92264 Use DISPLAY_CATEGORY_ALL_INCLUDING_DISABLED for getDisplays
Bug: 227459045
Bug: 221071695
Test: manual
Change-Id: I8c1587e05a30abf75a01a84ebda6522606abb99b
2022-06-09 12:12:57 +00:00
Andy Wickham
ce647cc9ae Fixes issue where personal/work tabs could overlap search reasults.
When you start searching, the header is set to "collapsed" mode.
This normally hides the tabs indirectly if the height of the
header changed as a result, which happens if there is a prediction
row because the previous height is non-zero but is considered 0
after the collapse. However, if there was no prediction row, the
height was already 0, so the header was not re-initialized, which
is where the tab visibility takes effect.

Fix: 235299156
Test: Manually with and without app predictions enabled and
keyboard default on/off.

Change-Id: I192a700ac83776f30d3207f92636113bdb8a86c5
2022-06-08 14:46:46 -07:00
Andy Wickham
5a476da75c Fixes Search bar padding when work profile not present.
The full padding was accounted for between the search bar and the
work/personal tabs, but only part of it was there when the tabs
were not shown.

Screenshot: https://screenshot.googleplex.com/8axV9QQ5Qy8MaUn.png

Test: Manually with and without work profile added. In the latter
case, nothing changed. The screenshot reflects the former case.
Fix: 233010483

Change-Id: I11b3c8cc814be48c25cbd9f454c3e122fff96fa0
2022-06-08 13:32:41 -07:00
Becky Qiu
68a63e0b73 [Toast] Add a new latency type as FAIL.
Bug: 227696256
Test: no need to test.
Change-Id: I84cdbbfc605494062ba07d79bee96a006e7ea7a5
2022-06-08 19:00:33 +00:00
Becky Qiu
fe83fce489 Merge "[Toast] Add query_length for launcher latency logging." into tm-dev 2022-06-08 18:02:07 +00:00
Jon Miranda
2dac4f7f4c Merge "Pause expensive view updates before setting hw/sw layers." into tm-dev 2022-06-08 00:15:35 +00:00
Jon Miranda
6d3d7b1a2e Merge "Play fallback animation if icon in All Apps is covered by header." into tm-dev 2022-06-08 00:14:45 +00:00
James O'Leary
7bbf0b5e42 Merge "Update notification dot from T90 to T80" into tm-dev 2022-06-07 22:39:46 +00:00
Becky Qiu
a61a2cc962 [Toast] Add query_length for launcher latency logging.
Bug: 235249335
Test: no test needed.
Change-Id: Ia16c241a2eb3cb50a17713216182192f7902be9e
2022-06-07 14:20:52 -07:00
Andy Wickham
2d5bda75fc Merge "Enables keyboard option by default." into tm-dev 2022-06-06 21:45:17 +00:00
James O'Leary
ede514f275 Update notification dot from T90 to T80
Bug: 213314628
Test: work with UX to pick something darker than T80, present options
from T70 to T90, get feedback to use T80, verify at runtime via
screenshot dot is T80.

Change-Id: Ic5b47a419eb7697c22c4dd9ab849b8a00d73be9f
2022-06-06 16:01:45 -04:00
Hyunyoung Song
d4a79eb99b Merge "Fix IME hiding animation being delayed after unlocking the device" into tm-dev 2022-06-04 05:10:48 +00:00
hyunyoungs
d7a02ec7ae Fix IME hiding animation being delayed after unlocking the device
Hide keyboard with WindowInsetsController if could. In case
hideSoftInputFromWindow may ignored by input connection being finished
when the screen is off.

Bug: 233158645
Test: manual as steps:
  1.  Launch AllApps.
  2.  Tap search bar to invoke IME.
  3.  Screen off.
  4.  Screen on.
And all other keyboard hide cases such as back gesture, swipe down, etc.

Change-Id: I4154f38f94b6760a430f7510a650422020d8f59a
2022-06-03 14:19:45 -07:00
TreeHugger Robot
dd053e79e2 Merge "Adds helper methods in SearchUiManager and OnboardingPrefs." into tm-dev 2022-06-03 20:10:27 +00:00
Andy Wickham
9899468884 Enables keyboard option by default.
This just enables the setting for this feature. The default
value of the setting itself is still false.

Bug: 231433168
Test: Manually verified the option worked as expected, including
showing the user education when typing directly in All Apps
without tapping the QSB. Also verified previous features like
the snackbar to enable keyboard by default worked as before.

Change-Id: If423ae49436d1bf71579d52a99aea5ca3f2407dc
2022-06-02 16:51:01 -07:00
Andy Wickham
e20a35c7a3 Adds helper methods in SearchUiManager and OnboardingPrefs.
Bug: 233656600
Test: With topic change.
Change-Id: I9ddb910e94d389ec682617ce874fff6a2ce703c8
2022-06-02 16:44:16 -07:00
Jon Miranda
f58db150af Play fallback animation if icon in All Apps is covered by header.
Bug: 230707341
Test: enter all apps
      scroll so icon is covered by header
      open app, swipe back to return to all apps
      confirm fallback animation plays

      repeat but scroll so icon is not covered by header
      confirm normal animation plays

Change-Id: I56cda07f3eccc11e8909f7fb28210f65cc6e0f1d
2022-06-02 16:06:37 -07:00
Jon Miranda
46ecc0ca08 Pause expensive view updates before setting hw/sw layers.
Prevents widgets from updating mid animation.
Also pauses view updates during app close animation.

Bug: 220939231
Bug: 230617085
Bug: 226171754
Test: manual

Change-Id: I0138d57e6a7b2c22fd9a029e971b3e27c7e9f22e
2022-06-02 12:21:01 -07:00
Sunny Goyal
c2c1fdad54 Merge "Fixing wrong diffing logic in AdapterItem" into tm-dev 2022-06-02 16:45:31 +00:00
Stefan Andonian
0c809bbe67 Merge "Give FloatingIconView a copy of, rather than, the original icon." into tm-dev 2022-06-02 15:33:45 +00:00
Sunny Goyal
959a9f5444 Fixing wrong diffing logic in AdapterItem
Bug: 232932442
Test: Verified on device
Change-Id: I87eae4c238a7b2d9749859307d253822916bb31c
2022-06-01 15:02:39 -07:00
Luca Zuccarini
a6c36eb4d0 Merge "Update the workspace and background scale interpolators." into tm-dev 2022-06-01 16:51:42 +00:00
Luca Zuccarini
46338b09e3 Merge "Remove workspace fade from ALL_APPS > NORMAL transition." into tm-dev 2022-06-01 09:35:20 +00:00
James O'Leary
202ac414bf Merge "Revert "Update app icon shadows"" into tm-dev 2022-06-01 01:29:34 +00:00
James O'Leary
dac75aef3b Revert "Update app icon shadows"
Revert submission 17935777-lightDarkDetection

Reason for revert: Talked with Adam Cohen, white shadows are controverstial and we'd need to escalate to senior UX if we want to ship. James would rather do it in U timeframe, will flag to UXers we work with that the opportunity to escalate is available if needed.
Reverted Changes:
Ia91221427:Update light/dark detection.
Ie12d3d054:Update app icon shadows
Icc88c3496:Update smartspace shadow

Change-Id: I8bc90e79073b8e088f2912fef2af4c6c99a33553
2022-05-31 21:12:38 +00:00
Stefan Andonian
2a7fbee0d4 Give FloatingIconView a copy of, rather than, the original icon.
Previously, if a user tapped on the clock icon then rapidly swiped back,
the clock icon would stop updating. This fixes that issue by avoiding
changes to the original icon's inner callback state while preparing and
displaying the FloatingIconView.

Bug: 188377749
Test: Locally enabled the second hand on the clock icon in addition to
the minute and hour hands, then verified that it was incrementing
correctly every second even after tapping then rapidly swiping back on
the clock icon.

Change-Id: Ib8eddb53de293194e5f9dce2fcafe8a481a6116c
2022-05-31 17:32:08 +00:00
Luca Zuccarini
a36af25ab7 Remove workspace fade from ALL_APPS > NORMAL transition.
This applies to phones only for the one-off transition (Home gesture or
swipe back. See before and after videos in the bug.

Bug: 234359814
Test: manual
Change-Id: I8f4fc9ec40687b641a721bd28b32fcf50514861f
2022-05-31 15:46:28 +00:00
Luca Zuccarini
7ee40a4190 Update the workspace and background scale interpolators.
The spec had a typo (EMPHASIZED_DECELERATE instead of
EMPHASIZED_ACCELERATE). In addition, we're now using LINEAR for the
background since the feel is smoother this way. Cleared with motion
(simonroberts@).

Bug: 232418544
Test: manual
Change-Id: I492dbe8008856d331836793177f1dd58c3265ca2
2022-05-30 09:47:40 +00:00
James O'Leary
2b6700a94c Merge "Update app icon shadows" into tm-dev 2022-05-27 19:44:39 +00:00
Winson Chung
1e211e0c03 Merge "Ensure page scrolls are initialized even for 0 children." into tm-dev 2022-05-27 17:42:22 +00:00
Brian Isganitis
deaf8985f6 Merge "Have separate DeviceProfile instances for taskbar and all apps windows." into tm-dev 2022-05-27 17:25:16 +00:00
Vinit Nayak
e8a4e0e2dc Merge "Limit starting bounds of swipe to animation start rect" into tm-dev 2022-05-27 02:52:36 +00:00
Winson Chung
359cb5d1b6 Ensure page scrolls are initialized even for 0 children.
Bug: 233713180
Test: Walk through CtsVerifier Lock Task UI flow as described in the bug
Change-Id: I49cb942a37a202b3e1b7ee00479d68402fcaf8b0
2022-05-26 23:38:48 +00:00
Vinit Nayak
f7b4e7f497 Limit starting bounds of swipe to animation start rect
* If the starting point is greater than that of the current
device's width (in either positive or negative direction),
reset the starting rect to be fullscreen task bounds
* More details at b/228829958#comment12

Fixes: 228829958
Test: Reboot device and swipe up from home immediately,
app doesn't fling from the side.
Tested with portrait and landscape launcher.
Tested with fake landscape launcher.

Change-Id: I6ea24e30e9de5716b7830f487b2ed63f56598c50
2022-05-26 14:43:26 -07:00
Andy Wickham
4ca247a6bb Merge "Moves Search results into a separate RV (take 2)." into tm-dev 2022-05-26 21:41:55 +00:00
Brian Isganitis
afd72f0726 Have separate DeviceProfile instances for taskbar and all apps windows.
We want to scale down the DeviceProfile for taskbar, but the all apps
window should rely on the original DeviceProfile.

Test: Manual
Fix: 232907361
Change-Id: Ia09f674ada9e445c1d7278fa94c536ea9de13ef9
Merged-In: Ia09f674ada9e445c1d7278fa94c536ea9de13ef9
(cherry picked from commit a9a78117c7)
2022-05-26 17:21:50 +00:00
Brandon Dayauon
5ba5c76c76 Merge "fixed scalable grid allApps to match home screen icons in largest display size" into tm-dev 2022-05-26 16:06:02 +00:00
Alex Chau
56e367b744 Merge "Fix taskbar icon flicker and jump when transforming to hotseat" into tm-dev 2022-05-26 09:24:23 +00:00
Abhilasha Chahal
d18c5085c5 Merge "Add type TYPE_ADD_TO_HOME_CONFIRMATION to AbstractFloatingView" into tm-dev 2022-05-25 18:31:59 +00:00
Alex Chau
179c289a6c Merge "Center drop target button alignment across devices" into tm-dev 2022-05-25 12:38:11 +00:00
Alex Chau
a2ecf75082 Merge "Only setup SecondaryDropTarget in onDragStart" into tm-dev 2022-05-25 12:38:11 +00:00
Alex Chau
158caf466c Only setup SecondaryDropTarget in onDragStart
- In LauncherAccessibilityDelegate.getsupportedActions, it calls supportsAccessibilityDrop on each button regardless if DropTargetBar is visible or not. SecondaryDropTarget.supportsAccessibilityDrop attempts to setupUi which causes measure/layout of DropTargetBar (as well as DragLayer) each time LauncherAccessibilityDelegate.getsupportedActions is called
- To avoid this, only setupUi in onDragStart. In other cases don't call setUi.
- Also fixed a bug that when accessibility is on, upon dragging a suggested app, the secondary action suddenly changed to Uninstall

Fix: 233383721
Test: Enable accessibility, observe no excessive amount of onLayout is called on DropTargetBar
Change-Id: I4d1a00d320cceed81fa1312b5fc5f1081637bea8
2022-05-25 11:04:42 +00:00
Alex Chau
e2e1509d55 Fix taskbar icon flicker and jump when transforming to hotseat
- Apply additional translation on TaskbarView to account for difference between taskbar icon to bottom spacing compared to hotseat icon to bottom spacing
- Call updateIconAlignment outside of synchronizeNextDraw's then block, which get run after the synchronization

Bug: 204850744
Test: manual
Change-Id: Id65842f506eb342105082649446eb694cd5c33a4
(cherry picked from commit 51da219869)
Merged-In: Id65842f506eb342105082649446eb694cd5c33a4
2022-05-25 10:53:29 +00:00