summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJon Miranda <jonmiranda@google.com>2017-08-14 12:35:53 -0700
committerJon Miranda <jonmiranda@google.com>2017-08-14 12:35:53 -0700
commitbb10fa8fc2f125245163a50bf6cd04ff03bdfac4 (patch)
tree32a35c43f15e609c3fde07c8d33364ff6849258e
parent56ad97cf2de97ea91fff6e6ab5165791f272f5ba (diff)
downloadandroid_packages_apps_Trebuchet-bb10fa8fc2f125245163a50bf6cd04ff03bdfac4.tar.gz
android_packages_apps_Trebuchet-bb10fa8fc2f125245163a50bf6cd04ff03bdfac4.tar.bz2
android_packages_apps_Trebuchet-bb10fa8fc2f125245163a50bf6cd04ff03bdfac4.zip
Fix bug where all apps scrollbar jumps.
The problem is that rowAppIndex is not being set when we swap in the new predictions. So when we call AllAppsRecyclerView#getCurrentScrollY, the "item.rowAppIndex == 0" clause returns true for every predicted app, which results in a larger than expected value. Bug: 64628968 Change-Id: I37ca660522b5a99666de2ea0b9c0aa8cd3732aca
-rw-r--r--src/com/android/launcher3/allapps/AlphabeticalAppsList.java1
1 files changed, 1 insertions, 0 deletions
diff --git a/src/com/android/launcher3/allapps/AlphabeticalAppsList.java b/src/com/android/launcher3/allapps/AlphabeticalAppsList.java
index ee2756f2c..43b56a4bb 100644
--- a/src/com/android/launcher3/allapps/AlphabeticalAppsList.java
+++ b/src/com/android/launcher3/allapps/AlphabeticalAppsList.java
@@ -362,6 +362,7 @@ public class AlphabeticalAppsList {
for (int i = 0; i < size; ++i) {
AppInfo info = apps.get(i);
AdapterItem appItem = AdapterItem.asPredictedApp(i, "", info, i);
+ appItem.rowAppIndex = i;
mAdapterItems.set(i, appItem);
mFilteredApps.set(i, info);
mAdapter.notifyItemChanged(i);