Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 93ad57bd authored by Matt Pietal's avatar Matt Pietal
Browse files

Remove overscroll on keyguard when swiping up then down 

This very specific set of gestures can sometimes cause the
notifications to overlap the lock icon/udfps icon after the
user stops dragging. For the migration effort, it was safer
to remove this small overscroll for the drag up/down case
then continue to support it. There is no loss of functionality.

Fixes: 329399801
Test: Swipe up slightly on lockscreen, then down
Flag: ACONFIG com.android.systemui.migrate_clocks_to_blueprint
TEAMFOOD

Change-Id: Iba86a6f233fc563f556825a14d06fc2de394fff6
parent 70f58835

packages/SystemUI/src/com/android/systemui/shade/NotificationPanelViewController.java

+6 −1
Original line number Diff line number Diff line
@@ -3991,7 +3991,12 @@ public final class NotificationPanelViewController implements ShadeSurface, Dump 
                mExpandLatencyTracking = false;

            }

            float maxPanelHeight = getMaxPanelTransitionDistance();

            if (mHeightAnimator == null) {

            if (mHeightAnimator == null && !MigrateClocksToBlueprint.isEnabled()) {

                // MigrateClocksToBlueprint - There is an edge case where swiping up slightly,

                // and then swiping down will trigger overscroll logic. Even without this flag

                // enabled, the notifications can then run into UDFPS. At this point it is

                // safer to remove overscroll for this one case to prevent overlap.



                // Split shade has its own overscroll logic

                if (isTracking()) {

                    float overExpansionPixels = Math.max(0, h - maxPanelHeight);