You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Here's how to recreate a bug I spotted. (See the Current behavior gif below.)
Start the demo app.
Long press any tile (for example tile 0 hereafter) to switch to dragselect mode.
Important: do not perform any drag gesture, just long press tile 0 till it becomes red.
Without touching anything else click on tile 0 again.
This click should deselect tile 0 (make it white again) but right now it does not happen.
If tile 0 is clicked on again (two times including step 3) now it is deselected.
Desired behavior should be being able to deselect at the step 3 right?
I think bug is in the following line:
break; needs to be changed to return false;
Right now the function onInterceptTouchEvent will return true in the case MotionEvent.ACTION_DOWN: and the user does not receive an onClick event for the ViewHolder of tile 0 .
Current behavior:
Desired behavior:
The text was updated successfully, but these errors were encountered:
thadikari
changed the title
Can not unselect immediately after a long press
Can not deselect immediately after a long press
Jan 5, 2019
Hi, thanks for the much useful library!
Here's how to recreate a bug I spotted. (See the Current behavior gif below.)
Important: do not perform any drag gesture, just long press tile 0 till it becomes red.
This click should deselect tile 0 (make it white again) but right now it does not happen.
Desired behavior should be being able to deselect at the step 3 right?
I think bug is in the following line:
DragSelectRecyclerView/library/src/main/java/com/michaelflisar/dragselectrecyclerview/DragSelectTouchListener.java
Line 200 in 9d8b02e
break;
needs to be changed toreturn false;
Right now the function
onInterceptTouchEvent
will return true in thecase MotionEvent.ACTION_DOWN:
and the user does not receive anonClick
event for theViewHolder
of tile 0 .Current behavior:
Desired behavior:
The text was updated successfully, but these errors were encountered: