commit dfc94fc05f3a7117d0a87c4aaeadb871724e55dc
author fbaron <fbaron@google.com> Mon Mar 27 17:51:06 2023 -0700
committer Federico Baron <fbaron@google.com> Tue Mar 28 00:52:35 2023 +0000
tree 1185e7d271190c65946c1916714ef9ca5ab01122
parent f296b3421da43f332e0a71964e93d79e9fd63b5a

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

src/com/android/launcher3/widget/picker/WidgetsTwoPaneSheet.java

diff --git a/src/com/android/launcher3/widget/picker/WidgetsTwoPaneSheet.java b/src/com/android/launcher3/widget/picker/WidgetsTwoPaneSheet.java
index de7497a..2199473 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.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 @@
     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 @@
             resetExpandedHeaders();
             mRightPane.removeAllViews();
             mRightPane.addView(mRecommendedWidgetsTable);
+            mRightPaneScrollView.setScrollY(0);
         });
         mSuggestedWidgetsContainer.addView(mSuggestedWidgetsHeader);
     }
@@ -249,6 +250,7 @@
                 };
                 mRightPane.removeAllViews();
                 mRightPane.addView(widgetsRowViewHolder.itemView);
+                mRightPaneScrollView.setScrollY(0);
             }
         };
     }