Merge "Cleanup ENABLE_TASKBAR_POPUP_MENU" into udc-dev
This commit is contained in:
committed by
Android (Google) Code Review
commit
a47bd20ce4
@@ -55,7 +55,6 @@ import com.android.launcher3.LauncherSettings;
|
||||
import com.android.launcher3.R;
|
||||
import com.android.launcher3.accessibility.DragViewStateAnnouncer;
|
||||
import com.android.launcher3.anim.Interpolators;
|
||||
import com.android.launcher3.config.FeatureFlags;
|
||||
import com.android.launcher3.dragndrop.DragController;
|
||||
import com.android.launcher3.dragndrop.DragDriver;
|
||||
import com.android.launcher3.dragndrop.DragOptions;
|
||||
@@ -188,12 +187,10 @@ public class TaskbarDragController extends DragController<BaseTaskbarContext> im
|
||||
|
||||
DragOptions dragOptions = new DragOptions();
|
||||
dragOptions.preDragCondition = null;
|
||||
if (FeatureFlags.ENABLE_TASKBAR_POPUP_MENU.get()) {
|
||||
PopupContainerWithArrow<BaseTaskbarContext> popupContainer =
|
||||
mControllers.taskbarPopupController.showForIcon(btv);
|
||||
if (popupContainer != null) {
|
||||
dragOptions.preDragCondition = popupContainer.createPreDragCondition(false);
|
||||
}
|
||||
PopupContainerWithArrow<BaseTaskbarContext> popupContainer =
|
||||
mControllers.taskbarPopupController.showForIcon(btv);
|
||||
if (popupContainer != null) {
|
||||
dragOptions.preDragCondition = popupContainer.createPreDragCondition(false);
|
||||
}
|
||||
if (dragOptions.preDragCondition == null) {
|
||||
dragOptions.preDragCondition = new DragOptions.PreDragCondition() {
|
||||
@@ -208,8 +205,7 @@ public class TaskbarDragController extends DragController<BaseTaskbarContext> im
|
||||
public void onPreDragStart(DropTarget.DragObject dragObject) {
|
||||
mDragView = dragObject.dragView;
|
||||
|
||||
if (FeatureFlags.ENABLE_TASKBAR_POPUP_MENU.get()
|
||||
&& !shouldStartDrag(0)) {
|
||||
if (!shouldStartDrag(0)) {
|
||||
mDragView.setOnAnimationEndCallback(() -> {
|
||||
// Drag might be cancelled during the DragView animation, so check
|
||||
// mIsPreDrag again.
|
||||
|
||||
+1
-2
@@ -32,7 +32,6 @@ import com.android.launcher3.BubbleTextView;
|
||||
import com.android.launcher3.LauncherSettings;
|
||||
import com.android.launcher3.R;
|
||||
import com.android.launcher3.accessibility.BaseAccessibilityDelegate;
|
||||
import com.android.launcher3.config.FeatureFlags;
|
||||
import com.android.launcher3.logging.StatsLogManager;
|
||||
import com.android.launcher3.model.data.ItemInfo;
|
||||
import com.android.launcher3.model.data.WorkspaceItemInfo;
|
||||
@@ -75,7 +74,7 @@ public class TaskbarShortcutMenuAccessibilityDelegate
|
||||
|
||||
@Override
|
||||
protected void getSupportedActions(View host, ItemInfo item, List<LauncherAction> out) {
|
||||
if (ShortcutUtil.supportsShortcuts(item) && FeatureFlags.ENABLE_TASKBAR_POPUP_MENU.get()) {
|
||||
if (ShortcutUtil.supportsShortcuts(item)) {
|
||||
out.add(mActions.get(NotificationListener.getInstanceIfConnected() != null
|
||||
? SHORTCUTS_AND_NOTIFICATIONS : DEEP_SHORTCUTS));
|
||||
}
|
||||
|
||||
@@ -170,10 +170,6 @@ public final class FeatureFlags {
|
||||
"ENABLE_MINIMAL_DEVICE", DISABLED,
|
||||
"Allow user to toggle minimal device mode in launcher.");
|
||||
|
||||
public static final BooleanFlag ENABLE_TASKBAR_POPUP_MENU = getDebugFlag(270392477,
|
||||
"ENABLE_TASKBAR_POPUP_MENU", ENABLED,
|
||||
"Enables long pressing taskbar icons to show the popup menu.");
|
||||
|
||||
public static final BooleanFlag ENABLE_TWO_PANEL_HOME = getDebugFlag(270392643,
|
||||
"ENABLE_TWO_PANEL_HOME", ENABLED,
|
||||
"Uses two panel on home screen. Only applicable on large screen devices.");
|
||||
|
||||
Reference in New Issue
Block a user