summaryrefslogtreecommitdiffstats
path: root/go
Commit message (Collapse)AuthorAgeFilesLines
* Reduces padding of the Overview shelf in no button mode.Andy Wickham2019-09-031-2/+3
| | | | | | | | | | | | Before: https://screenshot.googleplex.com/iw6baTCfoy2.png After: https://screenshot.googleplex.com/8CcsQvLhYDo.png Desired: https://b.corp.google.com/action/issues/139551306/attachments/26925873?download=false Test: Examined before and after padding and verified 3-button is unaffected. Fixes: 139551306 Change-Id: Ia416399cf25e7c6c2fb8aa84ad20e7ad74dcd31b (cherry picked from commit ddc9622e92fb5a4b211a5942ec679d195dc10d19)
* show dot in deep shortcuts when notification contains exactly identicalPinyao Ting2019-08-051-12/+0
| | | | | | | | set of person Bug: 132336512 Change-Id: I975524e28168c10a186cdc24b188c161faf433cf Merged-In: I975524e28168c10a186cdc24b188c161faf433cf
* Fill the log container as hotseat or workspace based on position.Becky Qiu2019-08-011-1/+1
| | | | | | | | | | | | | | | | | | | Test: manual Bug: 137953006 Swipe from workspace: 07-30 15:15:44.031 9779 9779 D UserEvent: action:FLING direction=UP direction=UP 07-30 15:15:44.031 9779 9779 D UserEvent: Source child:WORKSPACE id=0 span(951,1313) parent:WORKSPACE id=0 07-30 15:15:44.031 9779 9779 D UserEvent: Destination child:ALLAPPS 07-30 15:15:44.031 9779 9779 D UserEvent: Elapsed container 826 ms, session 9361 ms, action 0 ms Swipe from hotseat: 07-30 15:15:46.010 9779 9779 D UserEvent: action:FLING direction=UP direction=UP 07-30 15:15:46.010 9779 9779 D UserEvent: Source child:HOTSEAT id=0 span(786,1908) parent:WORKSPACE id=0 07-30 15:15:46.010 9779 9779 D UserEvent: Destination child:ALLAPPS 07-30 15:15:46.010 9779 9779 D UserEvent: Elapsed container 1139 ms, session 11339 ms, action 0 ms Change-Id: I959528889fda778efc569bb59e7f44d3bd5b81bc
* Ensure each dot contains only relevant notification to the shortcutPinyao Ting2019-07-311-2/+5
| | | | | | | | | | | | | | and displays system shortcut when long click on deep shortcut. In this CR we 1) for each shortcut, we filters the notifications to ensure we only get notifications with identical shortcutId. 2) allow system shortcut to be displayed when user long click on a DeepShortcut Bug: 132336512 Change-Id: Idc9eaed55e900ed4027a43ee9c3fd7dc6f4480b2 Merged-In: Idc9eaed55e900ed4027a43ee9c3fd7dc6f4480b2
* show dot on shortcut when incoming notification contains exactly thePinyao Ting2019-07-301-0/+6
| | | | | | | | same shortcut id Bug: 132336512 Change-Id: Iddf4cfe8ad60c12e8de8b171bed392f1bb0a6761 Merged-In: Iddf4cfe8ad60c12e8de8b171bed392f1bb0a6761
* Using the first frame delay based on the display refresh rate instead ofSunny Goyal2019-07-241-2/+3
| | | | | | | | | hardcoding it to 16ms > Creating a utility class for caching display property changes Bug: 128940249 Change-Id: I6f9a214548de65bd1c8530508d665ee88312da4a
* Fix waiting for launcher initializationvadimt2019-07-181-1/+1
| | | | | | Bug: 137836033 Change-Id: Ifc8ce867abd85eaeacd09a2b39636ecc4109c046
* Correctly restarting Launcher from OOP testsvadimt2019-07-111-0/+4
| | | | | | | | Waiting for touch interaction service; waiting for package-restart events. Bug: 136215685 Change-Id: I0c31c09fe3a58b898168dfdb4e4d23757b94a47c
* Using a higher priority executor than BackgroundExecutor for various recents ↵Sunny Goyal2019-07-111-0/+5
| | | | | | | | tasks BackgroundExecutor is also using thread pool executor where the order is perserved Change-Id: Ieef8825599f35fe22da3e9adb1270c5525449d62
* Resetting UI state when the last has launchedSunny Goyal2019-06-282-0/+8
| | | | | Bug: 135038270 Change-Id: Ie0af9c7d47f69b32caedd72f98ed1706bdbef5fa
* Finish recents animation when starting a new activityWinson Chung2019-06-261-0/+2
| | | | | | | | | - Defer starting the activity when an activity is paused, and finish the current animation to trigger launcher to be resumed Bug: 132811175 Test: Swipe up and launch a new app Change-Id: I78b76800052512eb93f69ccf0523f4d752a82ece
* Fix NPE in OverviewComponentObserver constructor while accessing activityInfo.Rajeev Kumar2019-06-251-8/+43
| | | | | | | Bug: 132461400 Test: Manual tests Change-Id: Icee3730454db8129e597714c147c2cbaddc70987 (cherry picked from commit 219208a0f53e8d5f613b9d197cf4072685dd9c0f)
* Some improvements to home to overview transition for 0 button modeTony Wickham2019-06-192-19/+40
| | | | | | | | | | | - Peek in overview further, and with more overshoot - Fade out and scale down workspace faster while swiping up - Scale and translate workspace slower when letting go to enter overview (so it doesn't zoom out at warp speed) - Fade in all apps shelf sooner Bug: 132455160 Change-Id: Ieafad0ccf9bb587889bc35d536627661db10e358
* Gracefully handle failed task launches during quick switchTony Wickham2019-06-121-0/+5
| | | | | | | | When quick switching from home or overview, go to OverviewState if the task launch failed. Otherwise we get stuck in BackgroundAppState. Bug: 135038270 Change-Id: I42785c261cef0df95666bc106ec5ca6ef0553cc7
* Fix getting stuck in BackgroundAppState for 2-button modeTony Wickham2019-05-311-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | When ending in recents, we reapply the state - therefore, it's important to make sure we are in OverviewState before onSwipeUpComplete(). This is done by mLauncherTransitionController, which sets OverviewState on end. We already force mLauncherTransitionController to end before calling onSwipeUpComplete(), but in this case we were calling cancel() and setting mLauncherTransitionController = null, which meant we could never call end() on it. Instead, we should always call end() if we set it to null. Also ensure mLauncherTransitionController is created even if the gesture is completed, if an existing controller isn't already running. This can happen if you swipe up quickly enough that we get onGestureEnd before launcher is drawn, and in that case we still want the launcher component to animate once its ready. This is even more important for 2-button mode, because again, we rely on mLauncherTransitionController to set the state to OverviewState before we reapply it. Finally, clarified some methods by renaming "swipeUp" to "swipeUpToRecents". Bug: 132757019 Change-Id: Ieb24a4f36a39780e5d64d7bc312791608db474d1
* Recents Go: Conform to device default styleKevin2019-05-223-2/+5
| | | | | | | | | | | | | System apps in Q should attempt to conform to device style, so naturally Recents Go should also conform to this so that text changes with the device style changing. This CL ensures all text views change font based off system theming title fonts. Bug: 131839392 Fix: 131839392 Test: Change device theme body font, see that change is propogated to Recents Go app texts, clear all, and empty recents message Change-Id: Ib85ff49383221ef18ed8aa9e53eff129129e37c2
* Merge "Scale down thumbnail with app surface" into ub-launcher3-qt-devTreeHugger Robot2019-05-142-4/+41
|\
| * Scale down thumbnail with app surfaceKevin2019-05-102-4/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | The recents Go app to overview transition has app scale down to a thumbnail but normally covers the thumbnail. However, apps with transparency will be semi-visible and will allow the user to see the thumbnail in the back at its final size. Instead, we should fit the thumbnail to the surface so that they both scale down at the same time. Bug: 132458092 Test: Go to app with transparency, scale down Change-Id: Iaebeaaf2ddcfc86fd4f55ef9d8c3f19583947c48
* | Merge "Fallback recents should wait for remote anim ready" into ↵TreeHugger Robot2019-05-131-0/+29
|\ \ | | | | | | | | | ub-launcher3-qt-dev
| * | Fallback recents should wait for remote anim readyKevin2019-05-131-0/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Hook into the remote animation API on the recents view for the fallback recents code path. This ensures the views are attached before attempting the layout animation. Bug: 132112131 Test: Set separate default launcher, go to recents from app, views animate correctly Change-Id: Id73f18e5f864e1970f954c7ec633e25e466d832e
* | | Fixing insets mapping in 3-button and 2-button modeSunny Goyal2019-05-131-0/+6
| | | | | | | | | | | | | | | Bug: 131360075 Change-Id: If6e3a4fbb011fc313efeb91686a9d787761862c5
* | | Merge "Add bottom margin to recents Go again" into ub-launcher3-qt-devTreeHugger Robot2019-05-131-5/+1
|\ \ \
| * | | Add bottom margin to recents Go againKevin2019-05-101-5/+1
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | Navigation bar insets are not always transparent and shouldn't be used for margin. So we put the margin back to get a bit more space from the nav bar. Bug: 132461256 Test: Manual test, see margins Change-Id: Id92e8b80a0d2c18c0a603942600366e355649ea8
* | | Revert "Implement scrim method"Hyunyoung Song2019-05-111-5/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit d849ef084c180c393c7f3c69f6f00a01358a644f. Reason for revert: not needed I was too eager to merge this CL. It turns out all I need is these three constants and doesn't actually need the aidl change. public static final int MAIN_COLOR_LIGHT = 0xffdadce0; public static final int MAIN_COLOR_DARK = 0xff202124; public static final int MAIN_COLOR_REGULAR = 0xff000000; Change-Id: Iafd539a06455ba0878a6d3750f3c7194f8919fbf
* | | Implement scrim methodLucas Dupin2019-05-111-0/+5
| | | | | | | | | | | | | | | | | | | | | Bug: 130451254 Test: make Change-Id: Ifd66f9e53dd86d21fe66e4617cc4b209fecd7328 (cherry picked from commit 904c429b9998c96476ab5026bc04e7242d559f8d)
* | | Merge "Add task launch logging to Recents Go" into ub-launcher3-qt-devTreeHugger Robot2019-05-112-5/+15
|\ \ \
| * | | Add task launch logging to Recents GoKevin2019-05-092-5/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add logging points for launching tasks in Recents Go to match up with non-Go Recents. Bug: 131840601 Test: Log calls to confirm writing to stats log on task launch Change-Id: I3d7b1ed30f99ce9f551c3481628ca81540e366b5
* | | | Merge "Add basic home to recents anim for fallback recents" into ↵TreeHugger Robot2019-05-102-1/+53
|\ \ \ \ | | | | | | | | | | | | | | | ub-launcher3-qt-dev
| * | | | Add basic home to recents anim for fallback recentsKevin2019-05-102-1/+53
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For users/OEMs who ship with a different default launcher, we should provide at least a basic animation from home to recents. This creates a simple cross-fade animation if coming from a separate home activity. Bug: 132449252 Test: Have different default launcher, go to home, go to recents Change-Id: I5ac7b0eedbcdf1e020c31d9cf120887a4bb3826a
* | | | Merge "Fix missing early return." into ub-launcher3-qt-devTreeHugger Robot2019-05-101-0/+1
|\ \ \ \ | |/ / / |/| | |
| * | | Fix missing early return.Kevin2019-05-101-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix a missing early return for when we don't have the bottom view laid out on remote animation start. Test: Builds Change-Id: Iba0d5097ce59d017dc43030552f2171c5e6e4fb1
* | | | Merge "Refactor lambda to use listener instead" into ub-launcher3-qt-devTreeHugger Robot2019-05-102-30/+63
|\ \ \ \ | | |_|/ | |/| |
| * | | Refactor lambda to use listener insteadKevin2019-05-102-30/+63
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We should not use an in-line lambda here as we allocate a new RemoteAnimationProvider object to do so which hooks into the already existing AppToOverviewAnimationProvider. This can be a problem if AppToOverviewAnimationProvider overrides methods as we would not use the overridden methods but instead the default ones. Instead, we use the same AppToOverviewAnimationProvider object but hook into the activity ready and window animation calls with a listener to add on the logging logic. Bug: 132112131 Test: Manual test; app to overview functions as before Change-Id: Ie70541691ed420c33770d6ed4f54d9555374dc0a
* | | | Update scrim if insets are added later.Kevin2019-05-101-1/+12
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If insets are set after the activity requests the foreground scrim to be shown, the scrim will not be correctly added. This is an issue for fallback recents which requests the foreground scrim right after initialization. To fix this, we save the request to show the scrim in a local variable and check this on inset updates to determine whether we should show the scrim or not. Bug: 131853975 Test: Fallback recents now has foreground scrim added correctly Change-Id: I15a19a3184bbc97a20fbcbba2106fd7221410df0
* | | Merge "Set "usingRemoteAnim" for fallback recents" into ub-launcher3-qt-devTreeHugger Robot2019-05-101-1/+1
|\ \ \
| * | | Set "usingRemoteAnim" for fallback recentsKevin2019-05-101-1/+1
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | Ensure fallback recents uses the correct layout animation by hooking into usingRemoteAnim API. Bug: 132112131 Test: Manual test Change-Id: I9bb96dbf436018e5488ede0c5ca48af787180aec
* / / Only do content fill anim after layout animKevin2019-05-101-6/+16
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | The new remote animation may start after the tasks are loaded and we start the content fill animation. This means the layout happens after and the content fill animation will conflict with the layout animation. To fix this, we should only do the content fill animation after the layout animation as intended. If the tasks load before we start the layout animation, we should just directly update to the latest UI without animating as the views won't be visible before the layout animation starts. Bug: 132381412 Test: Manual test Change-Id: Ibb55d8ef4499260ffa4bef5c6bba605ffdc85da7
* | Merge "Fix anim done callback not being called sometimes" into ↵TreeHugger Robot2019-05-101-1/+4
|\ \ | | | | | | | | | ub-launcher3-qt-dev
| * | Fix anim done callback not being called sometimesKevin2019-05-091-1/+4
| |/ | | | | | | | | | | | | | | | | | | | | We should call dispatchFinishedWhenDone in endAnimations instead of calling dispatchAnimationsFinished as the former will run the listener. However, we only do this if we were running animations in the first place. Bug: 132285006 Test: Build and manual test endAnimations() call Change-Id: I31afdc4c9e3ef5f56eda6178569d5c264c16ae78
* | Ending pending anim should set to end values.Kevin2019-05-091-0/+2
| | | | | | | | | | | | | | | | | | | | Pending animations have not started but when ending animations, we should ensure they are set to their end values based off the documentation for ItemAnimator's endAnimations and endAnimation. Bug: 132285006 Test: Build using endAnimations and manual test Change-Id: I3039a4f192d68a0ead700cd81675b3e7b4ef51f9
* | Move shared end pending anim logic for reusabilityKevin2019-05-091-26/+17
| | | | | | | | | | | | | | | | Some of the logic in endAnimations and endAnimation is shared and should be moved to a common helper method. Test: Builds and functions as before Change-Id: I4973d50ad8778bb397938b5d65d9f95f48ff145b
* | Merge changes from topic "polish-app-recents-anim" into ub-launcher3-qt-devTreeHugger Robot2019-05-105-114/+265
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | * changes: Polish app => recents animation (5/5) Polish app => recents animation (4/5) Polish app => recents animation (3/N) Polish app => recents animation (2/N) Polish app => recents animation (1/N)
| * | Polish app => recents animation (5/5)Kevin2019-05-092-0/+77
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add the part of the animation where task views fade in from bottom to top. For the app animating in, we only fade in the label and the icon as the snapshot needs to be visible as the app window scales down to it. Bug: 132112131 Test: Manual test animation Change-Id: Ia3fb98fde0b14fa4f72b53a1941cc2ee6b9f2294
| * | Polish app => recents animation (4/5)Kevin2019-05-081-20/+64
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add the vertical ease in component to the app => recents animation. In addition to animating over all task views, this should also animate over the closing app surface so that the surface movement matches the thumbnail as it moves to cover it. As a result, we take in the transformation matrix thats passed in to the app surface and apply the vertical motion to it. Bug: 132112131 Test: Manual Change-Id: I77c6716082af6ee722b96a2c362e16030cec8b8b
| * | Polish app => recents animation (3/N)Kevin2019-05-082-35/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This CL takes the current scale down animation and updates its interpolators and durations to match the UX spec. In addition, we move the core logic to a helper method as the whole remote animation will have other animations as well in the future. Bug: 132112131 Test: Manual Change-Id: I68b1d623b1b2c980dc34e41bbb5460cc021b5887
| * | Polish app => recents animation (2/N)Kevin2019-05-072-104/+106
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This CL moves the core "app to overview" animation logic into the main view as the newer animation would need to animate over several of the main view's children and so the animation logic needs knowledge of those children. Bug: 132112131 Test: Build and test, functions as before Change-Id: I9cbde55a582bee62e0a97e38c5fdf1d5841502db
| * | Polish app => recents animation (1/N)Kevin2019-05-073-1/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As part of building out a better remote app to overview animation, we first separate out the logic so that we only do the default layout animation when coming from a Launcher state (i.e. from home or all apps) and only then, so coming from an app no longer leads to the tasks animating. We do this with a simple flag that indicates that recents will be using a remote animation. Bug: 132112131 Test: Go to app, go to recents. No layout animation. Test: Go to home, go to recents. Layout animation works as normal Change-Id: I30988d944571fd5317d0c9d13e2a99b167c1291b
* | | Merge "Use Animator cancel instead of end" into ub-launcher3-qt-devTreeHugger Robot2019-05-091-1/+3
|\ \ \
| * | | Use Animator cancel instead of endKevin2019-05-091-1/+3
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Animtor's cancel() skips some animator init logic that end() does so we'd like to use that instead. The on end callback will still set the animation value to its target end value so this doesn't affect correctness. Bug: 132285006 Test: Build and manual test endAnimations Change-Id: I450a888c7b37a7dc74f24035d9abc5bda1cbbbd8
* | | Merge "Fix fallback recents not updating list from app" into ub-launcher3-qt-devTreeHugger Robot2019-05-091-2/+2
|\ \ \ | |/ / |/| |