commit e8b7fb0ca0453d04dee4cb581d7938a3caa07f11
author Ram Parameswaran <rampara@google.com> Thu Aug 27 15:47:44 2020 -0700
committer Ram Parameswaran <rampara@google.com> Thu Aug 27 23:17:13 2020 +0000
tree 34fcee9488f020d492cff6605668de1a3d9f00dd
parent f3b5162995f17ed0d98d7714909ef4a0b55e7169

Fix pageDown scroll distance

Calculation when last item is not partially visible does not take into account padding from toolbar.

Fixes: 166675330
Test: Manual

Change-Id: Id0e6e97e0166a9567187bc8326c71d22ffd925ca

car-ui-lib/car-ui-lib/src/main/java/com/android/car/ui/recyclerview/DefaultScrollBar.java

diff --git a/car-ui-lib/car-ui-lib/src/main/java/com/android/car/ui/recyclerview/DefaultScrollBar.java b/car-ui-lib/car-ui-lib/src/main/java/com/android/car/ui/recyclerview/DefaultScrollBar.java
index b1e8872..8c047ac 100644
--- a/car-ui-lib/car-ui-lib/src/main/java/com/android/car/ui/recyclerview/DefaultScrollBar.java
+++ b/car-ui-lib/car-ui-lib/src/main/java/com/android/car/ui/recyclerview/DefaultScrollBar.java
@@ -348,7 +348,8 @@
             if (child.getHeight() > screenSize) {
                 if (orientationHelper.getDecoratedStart(child) > 0) {
                     // Child view top is entering screen. Align its top with parent top.
-                    scrollDistance = orientationHelper.getDecoratedStart(child);
+                    scrollDistance = orientationHelper.getDecoratedStart(lastChild)
+                            - orientationHelper.getStartAfterPadding();
                 } else if (screenSize < orientationHelper.getDecoratedEnd(child)
                         && orientationHelper.getDecoratedEnd(child) < 2 * screenSize) {
                     // Child view bottom is about to enter screen - its distance to parent bottom