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

LongClick #36

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

LongClick #36

wants to merge 3 commits into from

Conversation

cinita-ms
Copy link

when onLongClick, setSwipeEnable false.

@mcxtzhang
Copy link
Owner

Thanks for your pull request.Can you explain more about the scene of this pull request?

@cinita-ms
Copy link
Author

写侧滑时参照了您的思路,在这个情况下发现的,如果想要长按后显示多选状态,那我需要长按后禁掉侧滑功能,于是我在onLongCLick 中设置 isSwipeEnable 为false并且此时的isTouching为true, 此时onLongClick 事件优先于 dispatchTouchEvent.ACTION_UP和ACTION_CANCEL, 但isSwipeEnable 为false之后,自己写的dispatchTouchEvent.ACTION_UP和ACTION_CANCEL就不会执行,即isTouching不会被重置为false。这样导致view接下来的所有事件因为isTouching为true被returen。
总结,我目前想到简单的方法是,如果在dispatchTouchEvent.ACTION_UP和ACTION_CANCEL之前setSwipeEnable为false, 需要resetIsTouching 为false.

@cinita-ms
Copy link
Author

或者几个事件都分别判断isSwipeEnable, dispatchTouchEvent.ACTION_UP和ACTION_CANCEL中的isTouching = false 不参与判断,一定被重置。

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.

2 participants