summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Fixing two critical bugs in all apps transition controllerHyunyoung Song2016-09-202-5/+6
| | | | | | | | | | bug#1: if long press from page indicator region, you can pull up all apps even though the workspace is in overview state. bug#2: all apps should not intercept touch not only in overview mode but also in any modal state. Change-Id: I91c0a181f037f6c2465a983825da3a2da7e2920f
* Adding support for extending the touch target in the touchDelegateSunny Goyal2016-09-151-3/+26
| | | | | Bug: 31458312 Change-Id: Ic819bdede2776ff63ec17053cc1326415edc1ca0
* Change all apps caret target bounding boxHyunyoung Song2016-09-078-52/+61
| | | | | | | | | | | | | | | b/30400751 1) This also fixes the issue of apps drawer sliding up when page is swiped horizontally immediately above the hotseat area. 2) Disclaimer: this should be cp'ed to calgary. Hence, I tried to keep the diff small. Ideally, CaretDrawable animation handling should move to AllAppsHandleImageView via refactoring in follow up CL. Change-Id: I4d808ee2eeca53b40c0b4e4e0e364fef056e26c3 (cherry picked from commit 016eaeea99e163f0b5385cf660297e6c21ff21e5)
* Support user event logging for drag and dropHyunyoung Song2016-09-0411-58/+172
| | | | | | | | | | | | | b/30039490 Supported in this CL: - DnD: drag from container [WORKSPACE|HOTSEAT|FOLDER|ALLAPPS|WIDGETS|DEEPSHORTCUTS] drag to container [HOTSEAT,WORKSPACE,FOLDER,DROPTARGETS] - Source and target can be [FOLDER_ICON, ICON, DEEPSHORTCUT, WIDGET] - $ adb shell setprop log.tag.UserEvent DEBUG will turn on debugging Change-Id: I0b8b879b80e6dce85bbde6e7794f9e0677832603 (cherry picked from commit 59a238095e82fd02355f4cb53abe01655a50b051)
* Removing some reflection codeSunny Goyal2016-09-021-23/+8
| | | | | Change-Id: I8a0a707d69999e5f9911e7d8df7d4a659b3dfb8f (cherry picked from commit e2ec8a3cd52f2322016fcadbae8963a446e2b0b3)
* Merge "Blocking alpha animation if the initial and final state for the ↵Sunny Goyal2016-09-021-3/+46
|\ | | | | | | animation are both 0" into ub-launcher3-calgary
| * Blocking alpha animation if the initial and final state for the animationSunny Goyal2016-09-021-3/+46
| | | | | | | | | | | | | | | | are both 0 Bug: 30739854 Change-Id: I4f7c579dc3f64a4fed60d7b2f8ecd5f359e165ec (cherry picked from commit bf206123a8cfad9f1341b60ff9a138e64b30b6e3)
* | Merge "Bug fix in Alarm where it was not getting called correctly if the new ↵Sunny Goyal2016-09-022-4/+20
|\ \ | | | | | | | | | timeout was set that was smaller than the previously set timeout" into ub-launcher3-calgary
| * | Bug fix in Alarm where it was not getting called correctly if theSunny Goyal2016-09-022-4/+20
| |/ | | | | | | | | | | | | | | | | | | | | | | new timeout was set that was smaller than the previously set timeout > Using uptimeMillis in Alarm to avoid errors due to system time changes > Adding an extra check in Wait in case Thread.sleep eats up all the timeout Change-Id: Id1fac5e8fdb81a0c3c7a6a5e50586b2a2f180d06 (cherry picked from commit a2125e1d102034e1d638b5d1e973b5b77cfe1efd)
* | Merge "Fixing broken accessibility drag" into ub-launcher3-calgarySunny Goyal2016-09-022-7/+8
|\ \
| * | Fixing broken accessibility dragSunny Goyal2016-09-022-7/+8
| |/ | | | | | | | | | | | | | | | | | | | | > Allow touch events on hotseat while in accessible drag as drag now happens in spring loaded state. > Allow drop target buttons to ignore thershold check when in accessibility drag Bug: 30900444 Change-Id: I88274367983fc027b2ddde3a719ca943f4f48587 (cherry picked from commit 4583d09c0560a83bfd0320299482cc41ac460b0c)
* | Merge "Fixing overlay scroll state not getting reset properly when scrolling ↵Sunny Goyal2016-09-021-0/+1
|\ \ | | | | | | | | | is was cancelled" into ub-launcher3-calgary
| * | Fixing overlay scroll state not getting reset properly whenSunny Goyal2016-09-021-0/+1
| |/ | | | | | | | | | | | | | | scrolling is was cancelled Bug: 30834333 Change-Id: I3605cd2dd939057905d297b5a25ce8669d578125 (cherry picked from commit 4ff424abf91a24b2ef021e12ab4bed434f13cd40)
* / Landscape container padding update.Hyunyoung Song2016-09-021-3/+3
|/ | | | | | | b/30021487 Change-Id: I179825eba09f3c4c57175e8f6da49708dae1931c (cherry picked from commit 9d96fd579039d07a6a36887ae699907631014c8d)
* Use promisedIntent instead of intent when deep shortcuts are restored.Tony Wickham2016-08-305-11/+21
| | | | | | | | | | | This ensures that the intent has the package corresponding to the shortcut publisher, rather than a market intent. It also ensures that the intent has the EXTRA_SHORTCUT_ID attached. Bug: 31123204 Change-Id: I05d56396b629880322e915f52bfc0605b921b0b1 (cherry picked from commit fc02c1b446ee54561ac7351fb6ff0f8294785f0e)
* Fixing the hotseat import logicSunny Goyal2016-08-242-10/+41
| | | | | | | | | | | | | | | | | The import logic following the behavior: Improt everything and force run GridMigrationTask to automatically remove broken icons. This logic would fail for hotseat as the replacement happens before the GridMigrationTask, which will not replace the broken targets appropriately The cl changes some logic only for hotseat import > After import remove any broken icons/empty folders > When adding default icons, only add as much icons as required. Since GridMigrationTask uses weights, it sometimes removes imported icon, if the hotseat size becomes too large. Bug: 30909630 Change-Id: I6ca1f25dac81649794d81aaf82c3c38d1c918d91
* Checking for duplicate intent with proper source bounds.Sunny Goyal2016-08-241-1/+3
| | | | | | | | | | Since we are doing a equality check on the intent, items which already have the bounds attached to the intent will not get matched, causing duplicate icons Bug: 27176189 Change-Id: Ia9ed9120b04e9c234304995aed13777a5773119e (cherry picked from commit cbfe71dba38af1d1cd2d2372b649537d7d94610b)
* Cancel hotseat color animator before setting a new color.Tony Wickham2016-08-221-4/+16
| | | | | | | | | | | | | | | | | There was a race condition that caused the new color to not be applied. For example, here's how the hotseat would become transparent: 1 Launcher is loaded for the first time; as such, there is not yet a color defined for the hotseat, so we start animating to the provided default (Color.TRANSPARENT). Meanwhile we start the color extraction. 2 When the color extraction finishes, we set the hotseat to the new color. However, if launcher is paused at the time (perhaps some retail mode content is showing), then we don't animate the change. 3 If 2 happens before the animation in 1 is complete, the color from 2 will be overriden by subsequent animation frames and thus the hotseat remains transparent until the wallpaper changes. Bug: 30956221 Change-Id: Iddf72379b0162f1b32883ad26ce267473e172849
* Merge "Import translations. DO NOT MERGE" into ub-launcher3-calgaryTreeHugger Robot2016-08-161-1/+1
|\
| * Import translations. DO NOT MERGEBill Yi2016-08-151-1/+1
| | | | | | | | | | Change-Id: I96007581be40da18813dfcb8606bc6d8fed29e54 Auto-generated-cl: translation import
* | Merge "Updating version check to use SDK_INT" into ub-launcher3-calgarySunny Goyal2016-08-161-6/+1
|\ \
| * | Updating version check to use SDK_INTSunny Goyal2016-08-151-6/+1
| | | | | | | | | | | | Change-Id: Idb46053e7304bc8d0cab03a17a6fe73013e4f092
* | | [DO NOT MERGE] Set launch source target correctly for user event loggingHyunyoung Song2016-08-165-27/+75
| |/ |/| | | | | | | | | | | | | Before, everything is set to APP_ICON With this changed, pinned shortcuts are set to DEEPSHORTCUT Change-Id: I3e17de63f58693525236290ef5cb1f909f1d6098 (cherry picked from commit 8ce6063c4a5bd90810f0a21c946e5bbad3ce9de4)
* | Merge "Ignore clicks on shortcuts' drag handles." into ub-launcher3-calgaryTony Wickham2016-08-152-1/+38
|\ \
| * | Ignore clicks on shortcuts' drag handles.Tony Wickham2016-08-152-1/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When touching down within the drag handle's bounds, we remove the onClickListener temporarily (restored when touching down outside of the drag handle's bounds). Long clicks still start the drag. Also increased drag threshold from 12dp to 16dp. Bug: 30816665 Change-Id: I0b33dc34bf95c0532376f2f7cf50865fa50093de
* | | Merge "Log shortcuts open on ACTION_UP instead of in showForIcon()." into ↵Hyunyoung Song2016-08-123-36/+10
|\ \ \ | | | | | | | | | | | | ub-launcher3-calgary
| * | | Log shortcuts open on ACTION_UP instead of in showForIcon().Tony Wickham2016-08-103-36/+10
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - We only want to log when the container is opened and potentially used, not when a long press is followed by a drag-and-drop. - Also cleaned up code that was determining the container of the app icon, since LaunchSourceProvider.fillInLaunchSourceData() can do that instead (it's more robust and consistent). Bug: 30791570 Change-Id: I05b6750f26182fda8a9940ac66f1371c2d228ca9
* | | Import translations. DO NOT MERGEBill Yi2016-08-112-2/+2
| | | | | | | | | | | | | | | Change-Id: Iad4289a91834aa57c8458356a3f3b31be74c760b Auto-generated-cl: translation import
* | | force a blank spinSurendra Vispute2016-08-110-0/+0
| |/ |/|
* | Merge "Reloading workspace when the shortcuts permission changes" into ↵Sunny Goyal2016-08-113-5/+51
|\ \ | | | | | | | | | ub-launcher3-calgary
| * | Reloading workspace when the shortcuts permission changesSunny Goyal2016-08-103-5/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | > Check for permission on every onResume > If the permission is different than last known permission, reload and rebind workspace. Bug: 30789422 Change-Id: Idfa445815e29e2336505779545507d106b33a253
* | | Import translations. DO NOT MERGEBill Yi2016-08-101-1/+1
| | | | | | | | | | | | | | | Change-Id: Id72084a093ca622391d5c3af74f90e66499dea28 Auto-generated-cl: translation import
* | | Merge "Change drag from shortcuts to say "Cancel" instead of "Remove"" into ↵Tony Wickham2016-08-091-1/+1
|\ \ \ | |/ / |/| | | | | ub-launcher3-calgary
| * | Change drag from shortcuts to say "Cancel" instead of "Remove"Tony Wickham2016-08-091-1/+1
| |/ | | | | | | | | Bug: 30757836 Change-Id: I7d2d97898bd1feac3cd7ba15614c29a1e1b33841
* | Merge "Make padding consistent between shortcuts and the app icon." into ↵Tony Wickham2016-08-082-13/+20
|\ \ | | | | | | | | | ub-launcher3-calgary
| * | Make padding consistent between shortcuts and the app icon.Tony Wickham2016-08-052-13/+20
| |/ | | | | | | | | | | | | | | | | | | | | | | | | This change makes the padding consistent regardless of where the app is (e.g. folder vs workspace vs all apps) by ignoring the app's padding and adding our own to the shortcuts container. Also note that this padding is relative to the icon, excluding the text beneath it. So we also hide the text when the container opens downwards, and re-show it when the container closes. Bug: 30604007 Change-Id: I6e51c4983a8b5d495833f86e483ebaa229ed2099
* | Merge "Forcing drag layer to be non accessible. Setting a touch delegate ↵Sunny Goyal2016-08-083-0/+3
|\ \ | | | | | | | | | makes a view to become accessible, if its set to auto. This was making drayLayer accessible causing Talkback to speak out every item on the draglayer when window focus changes" into ub-launcher3-calgary
| * | Forcing drag layer to be non accessible.Sunny Goyal2016-08-043-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | Setting a touch delegate makes a view to become accessible, if its set to auto. This was making drayLayer accessible causing Talkback to speak out every item on the draglayer when window focus changes Bug: 30670702 Change-Id: Ifcba4c02b124166f9ca69c4244b9d20d7a7acaa5
* | | Fixing potential ConcurrentModificationExceptionSunny Goyal2016-08-082-2/+33
| | | | | | | | | | | | | | | | | | | | | | | | mBgDeepShortcutMap is only accessed on the background thread. But the same instance of list of values was getting passed to the UI thread, instead of being cloned. Change-Id: Ie7d0442d895304489ce9323ea872b9091d668ae5
* | | Import translations. DO NOT MERGEBill Yi2016-08-0780-162/+82
| | | | | | | | | | | | | | | Change-Id: Id2c49f09da28322e551dc6dc5b136cbf0c9bc943 Auto-generated-cl: translation import
* | | Merge "Pull down should work on search box, even if all apps scroll is not ↵Hyunyoung Song2016-08-053-6/+12
|\ \ \ | | | | | | | | | | | | at top. b/30295305" into ub-launcher3-calgary
| * | | Pull down should work on search box, even if all apps scroll is not at top.Hyunyoung Song2016-08-053-6/+12
| | | | | | | | | | | | | | | | | | | | | | | | b/30295305 Change-Id: I38bbe47d6e33929c550930c9f157c58fbea203ef
* | | | Merge "Fix hotseat flicker when opened all apps in landscape, rotate screen, ↵Hyunyoung Song2016-08-052-3/+2
|\ \ \ \ | | | | | | | | | | | | | | | pull down" into ub-launcher3-calgary
| * | | | Fix hotseat flicker when opened all apps in landscape, rotate screen, pull downHyunyoung Song2016-08-042-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | b/30595452 Change-Id: Ib634ffbc509c6b16ac58c184ccbca15ae936be14
* | | | | Merge "Change NavBar to transparent black b/30615471" into ub-launcher3-calgaryHyunyoung Song2016-08-057-10/+21
|\ \ \ \ \ | |_|_|_|/ |/| | | |
| * | | | Change NavBar to transparent blackHyunyoung Song2016-08-057-10/+21
| | |/ / | |/| | | | | | | | | | | | | | | | | | b/30615471 Change-Id: I7885e8a7fd63ee10c9b0107087046730e8acfde9
* | | | Merge "Use shortcut disabled message if available." into ub-launcher3-calgaryTony Wickham2016-08-053-0/+17
|\ \ \ \ | |/ / / |/| | |
| * | | Use shortcut disabled message if available.Tony Wickham2016-08-053-0/+17
| | | | | | | | | | | | | | | | | | | | Bug: 30681210 Change-Id: Ibb5fd58e5fc9557b63730bc1e3c3977133769466
* | | | Reduce distance to start deferred drag from shortcuts to 12dp.Tony Wickham2016-08-041-3/+2
|/ / / | | | | | | | | | Change-Id: Ida298760bb66b5894dab94ff4ccaa788c8604661
* | | Merge "Reset mActiveController to null in ↵Tony Wickham2016-08-041-0/+2
|\ \ \ | |_|/ |/| | | | | DragLayer.onInterceptTouchEvent()." into ub-launcher3-calgary