From 67def6cef16bf67aac3a791662dda41c085e24a7 Mon Sep 17 00:00:00 2001 From: Hyunyoung Song Date: Tue, 28 Jun 2016 16:38:18 -0700 Subject: [PATCH] All apps icons should become clickable b/29761088 b/29610290 > Workspace transition state was not being properly set which caused onClick to not allow apps to launch. Temporary solution: do not call the onLauncherTransitionPrepare inside AllAppsTransitionController. > Fixing another important bug which doesn't properly set workspace state to correct alpha value because hotseat background drawable is null when home is triggered from settings app after wallpaper is changed. The color extractor has not figured out how hotseat drawable color should be set to immediately after wallpaper change. Change-Id: I9f797dbda6c73c71816e5070ed4da37099b26a6d --- .../launcher3/allapps/AllAppsTransitionController.java | 5 ----- 1 file changed, 5 deletions(-) diff --git a/src/com/android/launcher3/allapps/AllAppsTransitionController.java b/src/com/android/launcher3/allapps/AllAppsTransitionController.java index 3157c133f2..21b655d7db 100644 --- a/src/com/android/launcher3/allapps/AllAppsTransitionController.java +++ b/src/com/android/launcher3/allapps/AllAppsTransitionController.java @@ -256,8 +256,6 @@ public class AllAppsTransitionController implements TouchController, VerticalPul mAppsView.getRevealView().setAlpha(mHotseatBackgroundAlpha); setProgress(mShiftRange); } else { - // TODO: get rid of this workaround to override state change by workspace transition - mWorkspace.onLauncherTransitionPrepare(mLauncher, false, false); View child = ((CellLayout) mWorkspace.getChildAt(mWorkspace.getNextPage())) .getShortcutsAndWidgets(); child.setVisibility(View.VISIBLE); @@ -419,9 +417,6 @@ public class AllAppsTransitionController implements TouchController, VerticalPul } public void finishPullDown() { - if (mHotseat.getBackground() != null) { - return; - } mAppsView.setVisibility(View.INVISIBLE); mHotseat.setBackgroundTransparent(false /* transparent */); mHotseat.setVisibility(View.VISIBLE);