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

AndroidClickFix #35

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

Conversation

ericbourque29
Copy link

Replaced UP and DOWN with TAP in the TouchObserver to avoid double events in adroid.

Replaced UP and DOWN with TAP in the TouchObserver to avoid double events in adroid.
@pspeed42
Copy link
Contributor

Which double events? You should get an event when you press and an event when you release... just like any other key or button.

@mrlem
Copy link

mrlem commented Dec 7, 2019

I suspect what @ericbourque29 was referring to is that in Android, not only will up and down events be fired, but also a tap event.

However, the TouchObserver doesn't consume those, so they will be passed on to the scene behind your UI. And from your scene, if you want to react to tap events, you will receive them all regardless whether or not they landed in your UI first.

(faced the same issue myself).

@mrlem
Copy link

mrlem commented Dec 7, 2019

Personally, I'd still consume UP and DOWN events, but also TAP events.

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.

3 participants