From dfc94fc05f3a7117d0a87c4aaeadb871724e55dc Mon Sep 17 00:00:00 2001 From: fbaron Date: Mon, 27 Mar 2023 17:51:06 -0700 Subject: [PATCH] Scroll to top when new header is selected in two pane widget picker Bug: 275441368 Test: Select a header and scroll, then select a different header and verify that it is scrolled to the top Change-Id: Ie11d074759e5336ee1bbdec000ab266cf43c2c31 --- .../launcher3/widget/picker/WidgetsTwoPaneSheet.java | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/com/android/launcher3/widget/picker/WidgetsTwoPaneSheet.java b/src/com/android/launcher3/widget/picker/WidgetsTwoPaneSheet.java index de7497a1a4..2199473e82 100644 --- a/src/com/android/launcher3/widget/picker/WidgetsTwoPaneSheet.java +++ b/src/com/android/launcher3/widget/picker/WidgetsTwoPaneSheet.java @@ -23,8 +23,8 @@ import android.view.LayoutInflater; import android.view.MotionEvent; import android.view.View; import android.view.ViewOutlineProvider; -import android.widget.FrameLayout; import android.widget.LinearLayout; +import android.widget.ScrollView; import androidx.annotation.NonNull; @@ -52,7 +52,7 @@ public class WidgetsTwoPaneSheet extends WidgetsFullSheet { private WidgetsListHeader mSuggestedWidgetsHeader; private LinearLayout mRightPane; - private FrameLayout mRightPaneScrollView; + private ScrollView mRightPaneScrollView; private WidgetsListTableViewHolderBinder mWidgetsListTableViewHolderBinder; private final ViewOutlineProvider mViewOutlineProviderRightPane = new ViewOutlineProvider() { @@ -156,6 +156,7 @@ public class WidgetsTwoPaneSheet extends WidgetsFullSheet { resetExpandedHeaders(); mRightPane.removeAllViews(); mRightPane.addView(mRecommendedWidgetsTable); + mRightPaneScrollView.setScrollY(0); }); mSuggestedWidgetsContainer.addView(mSuggestedWidgetsHeader); } @@ -249,6 +250,7 @@ public class WidgetsTwoPaneSheet extends WidgetsFullSheet { }; mRightPane.removeAllViews(); mRightPane.addView(widgetsRowViewHolder.itemView); + mRightPaneScrollView.setScrollY(0); } }; }