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

Commit 20b553d6 authored by William Xiao's avatar William Xiao
Browse files

Fix top and bottom swipes not working after touches on glanceable hub 

The bouncer and shade touch handlers coordinate carefully with the
hub's touch handling to allow for vertical swipes on the grid itself,
as long the hub iself is not consuming the touch. The hub relied on the
touch handlers to reset the nested scrolling state between touches.

However, if the touch is entirely consumed by the hub, such as when
tapping on a CTA tile button, scrolling on a widget has a list, or
moving the UMO, the touch state is not reset. This leads to gestures
that are not nested scrolls, such as swiping up from the very bottom
for bouncer or very top for shade to not work.

This CL resets the touch scroll state when a touch consumed by the hub
ends to fix these issues.

Bug: 368420327
Fixed: 368420327
Flag: com.android.systemui.communal_hub
Test: manually verified on device
Change-Id: Ide4e6812b7e60e5301f4e3c2d668fb337ffce86c
parent 010ee587

packages/SystemUI/compose/features/src/com/android/systemui/communal/ui/compose/CommunalHub.kt

+3 −0
Original line number Diff line number Diff line
@@ -310,6 +310,9 @@ fun CommunalHub( 
                                        it.changedToUp() || it.changedToUpIgnoreConsumed()

                                    }

                                )



                                // Reset state once touch ends.

                                viewModel.onResetTouchState()

                            }

                        }

                    }