From ef3dd6efe03c988be1fe980fbed75333017e6760 Mon Sep 17 00:00:00 2001 From: Adam Cohen Date: Tue, 14 Feb 2012 20:54:05 -0800 Subject: [PATCH] Fixing bug where widget preview wasn't fully updated upon drop -> The prebound widget is now added to the DragLayer to ensure that it goes through all necessary measure/layout/attach etc. Change-Id: I53ca32e88fcbc2211d2747c3b067816c0d518e30 --- .../android/launcher2/AppsCustomizePagedView.java | 3 +++ src/com/android/launcher2/Workspace.java | 15 +-------------- 2 files changed, 4 insertions(+), 14 deletions(-) diff --git a/src/com/android/launcher2/AppsCustomizePagedView.java b/src/com/android/launcher2/AppsCustomizePagedView.java index 2f8d128c0b..ff1118c2e2 100644 --- a/src/com/android/launcher2/AppsCustomizePagedView.java +++ b/src/com/android/launcher2/AppsCustomizePagedView.java @@ -570,6 +570,8 @@ public class AppsCustomizePagedView extends PagedViewWithDraggableItems implemen mLauncher.getAppWidgetHost().createView(mContext, mWidgetLoadingId, pInfo); info.boundWidget = hostView; mWidgetCleanupState = WIDGET_INFLATED; + hostView.setVisibility(INVISIBLE); + mLauncher.getDragLayer().addView(hostView); } }; post(mInflateWidgetRunnable); @@ -596,6 +598,7 @@ public class AppsCustomizePagedView extends PagedViewWithDraggableItems implemen AppWidgetHostView widget = info.boundWidget; int widgetId = widget.getAppWidgetId(); mLauncher.getAppWidgetHost().deleteAppWidgetId(widgetId); + mLauncher.getDragLayer().removeView(widget); } mWidgetCleanupState = WIDGET_NO_CLEANUP_REQUIRED; mWidgetLoadingId = -1; diff --git a/src/com/android/launcher2/Workspace.java b/src/com/android/launcher2/Workspace.java index 9274f81343..4e731a2edc 100644 --- a/src/com/android/launcher2/Workspace.java +++ b/src/com/android/launcher2/Workspace.java @@ -3003,23 +3003,11 @@ public class Workspace extends SmoothPagedView } } - // The following methods deal with animating an item from external drop - void onPreDraw(View v) { - if (v instanceof ViewGroup) { - ViewGroup vg = (ViewGroup) v; - for (int i = 0; i < vg.getChildCount(); i++) { - View child = vg.getChildAt(i); - onPreDraw(child); - } - } else if (v instanceof TextView) { - ((TextView) v).onPreDraw(); - } - } - public Bitmap createWidgetBitmap(PendingAddWidgetInfo widgetInfo) { int[] unScaledSize = mLauncher.getWorkspace().estimateItemSize(widgetInfo.spanX, widgetInfo.spanY, widgetInfo, false); View layout = widgetInfo.boundWidget; + mLauncher.getDragLayer().removeView(layout); layout.setVisibility(VISIBLE); int width = MeasureSpec.makeMeasureSpec(unScaledSize[0], MeasureSpec.EXACTLY); @@ -3030,7 +3018,6 @@ public class Workspace extends SmoothPagedView layout.measure(width, height); layout.layout(0, 0, unScaledSize[0], unScaledSize[1]); - onPreDraw(layout); layout.draw(c); c.setBitmap(null); return b;