Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Add
pointerType
to event #2760Add
pointerType
to event #2760Changes from 17 commits
880b134
9722fda
05fd2d3
5ef00c7
74d76dd
16e5ccb
baeac06
2013287
30c14e1
7c44264
5200224
7bf1e55
aa6830d
4695e7b
66c87ff
bc7dcfd
098f194
407bbbe
adbdf3c
846cfbf
711becb
ad7cbc0
fd4fe73
7e67f7b
ff9ad8c
f6b1ca5
316e70b
025f549
f74ce39
ab677e2
6a92b89
77ab5ce
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(just for clarification) I assume there is no need to set type of pointer on
MotionEvent.ACTION_HOVER_EXIT
right?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I don't think we need to set that on exit. The main idea was to set this variable when gesture receives first event (inside
onPointerDown
on web,touchesBegan
on iOS, etc.).Now I see that in
NativeViewGestureHandler
on iOS i callsetCurrentPointerType
also onTouchUp
methods. I will check if it is necessary.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As I said, I've tested that and it seems that we don't need most of those calls. I removed them in ab677e2. I've left one in
handleDragEnter
since I'm not sure when this method is called. If you know how to test it let me know, I'll be grateful, cc @j-piasecki.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
handleDragEnter
should be called when pointer moves into the view boundary without touching down. I'm not sure whether it requires to start drag inside the view (touch the view -> drag outside -> drag back inside) or not (touch outside the view -> drag into the view)There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, I've just checked that and it does require to start drag inside of view. Given that, I removed the line that sets
pointerType
inside that function in 77ab5ce.