Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Avoid touch lockup due to mQuickReturn == true #587

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ksperling
Copy link

This can happen when a DOWN event is triggered in the quickreturn area, and the finger then moved out of that area without triggering a drag (e.g. by moving vertically first and then across). If the menu is then closed in some other way (e.g. back key) the CustomViewAbove remains in mQuickReturn == true state and blocks all touches to the main content, making the app appear unresponsive or crashed to the user.

This can happen when a DOWN event is triggered in the quickreturn
area, and the finger then moved out of that area without triggering
a drag. If the menu is then closed in some other way (e.g. back key)
the CustomViewAbove remains in mQuickReturn == true state and blocks
all touches to the main content.
@trashkalmar
Copy link

This really helps, thank you!

@xu6148152
Copy link

yeah, this bug I have encountered
and I merge this commit to my library SlidingMenu

@ksperling

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants