diff options
author | Hyunyoung Song <hyunyoungs@google.com> | 2015-05-15 23:36:45 +0000 |
---|---|---|
committer | Android (Google) Code Review <android-gerrit@google.com> | 2015-05-15 23:36:46 +0000 |
commit | 2a1580ca7d24e4b5dbe815e831332d1dd1d3b392 (patch) | |
tree | 8bafeaf742b1705a3611b994f7da2af180ba2ea7 | |
parent | 312011094f0a00ba53fb5cc6e8d01d8ac90addf1 (diff) | |
parent | b790083b8f757e6d73b92460e6447acbf5490654 (diff) | |
download | android_packages_apps_Trebuchet-2a1580ca7d24e4b5dbe815e831332d1dd1d3b392.tar.gz android_packages_apps_Trebuchet-2a1580ca7d24e4b5dbe815e831332d1dd1d3b392.tar.bz2 android_packages_apps_Trebuchet-2a1580ca7d24e4b5dbe815e831332d1dd1d3b392.zip |
Merge "Transition from back to showing widget tray if shortcut/Widget configure activity was cancelled." into ub-launcher3-burnaby
-rw-r--r-- | src/com/android/launcher3/Launcher.java | 18 |
1 files changed, 14 insertions, 4 deletions
diff --git a/src/com/android/launcher3/Launcher.java b/src/com/android/launcher3/Launcher.java index 72472b9f4..a828b1a42 100644 --- a/src/com/android/launcher3/Launcher.java +++ b/src/com/android/launcher3/Launcher.java @@ -809,7 +809,7 @@ public class Launcher extends Activity } boolean isWidgetDrop = (requestCode == REQUEST_PICK_APPWIDGET || - requestCode == REQUEST_CREATE_APPWIDGET); + requestCode == REQUEST_CREATE_APPWIDGET || requestCode == REQUEST_CREATE_SHORTCUT); final boolean workspaceLocked = isWorkspaceLocked(); // We have special handling for widgets @@ -3447,10 +3447,19 @@ public class Launcher extends Activity /** * Sets up the transition to show the apps/widgets view. + * + * @return whether the current from and to state allowed this operation */ - private void showAppsOrWidgets(boolean animated, State toState) { - if (mState != State.WORKSPACE) return; - if (toState != State.APPS && toState != State.WIDGETS) return; + // TODO: calling method should use the return value so that when {@code false} is returned + // the workspace transition doesn't fall into invalid state. + private boolean showAppsOrWidgets(boolean animated, State toState) { + if (mState != State.WORKSPACE && mState != State.APPS_SPRING_LOADED && + mState != State.WIDGETS_SPRING_LOADED) { + return false; + } + if (toState != State.APPS && toState != State.WIDGETS) { + return false; + } if (toState == State.APPS) { mStateTransitionAnimation.startAnimationToAllApps(animated); @@ -3472,6 +3481,7 @@ public class Launcher extends Activity // Send an accessibility event to announce the context change getWindow().getDecorView() .sendAccessibilityEvent(AccessibilityEvent.TYPE_WINDOW_STATE_CHANGED); + return true; } /** |