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

Commit 5421eb69 authored by Josh Tsuji's avatar Josh Tsuji
Browse files

Fix manage not being dismissed by taps in the AV bounds. 

This actually isn't a touchable region issue - the expanded view steals touches from the stack if they're within the AV's vertical bounds so that back swipes across the side padding aren't intercepted by the stack, causing it to collapse.

We should be checking the x coordinate as well, since the stack *should* be able to intercept touches from the AV itself.

Fixes: 158674822
Test: manual
Change-Id: Ieb12e6d97a0c180139bcaa1634e0ce47899d7c72
parent ab6c4a76

packages/SystemUI/src/com/android/systemui/bubbles/BubbleExpandedView.java

+4 −1
Original line number Diff line number Diff line
@@ -351,7 +351,10 @@ public class BubbleExpandedView extends LinearLayout { 
            // ActivityView's vertical bounds. These events are part of a back gesture, and so they

            // should not collapse the stack (which all other touches on areas around the AV would

            // do).

            if (motionEvent.getRawY() >= avBounds.top && motionEvent.getRawY() <= avBounds.bottom) {

            if (motionEvent.getRawY() >= avBounds.top

                            && motionEvent.getRawY() <= avBounds.bottom

                            && (motionEvent.getRawX() < avBounds.left

                                || motionEvent.getRawX() > avBounds.right)) {

                return true;

            }