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

Add Touch Manager for improved onTouchEvent Handling #402

Closed
asm09fsu opened this issue May 19, 2016 · 3 comments
Closed

Add Touch Manager for improved onTouchEvent Handling #402

asm09fsu opened this issue May 19, 2016 · 3 comments
Milestone

Comments

@asm09fsu
Copy link
Contributor

Bug Report

After dealing with using the onTouchEvent callback, I propose we design an architecture for having a touch manager to give callbacks that allow developers to know if certain touch events are happening. Currently the following are thought of as important:

  • Taps
    • Single Tap
    • Double Tap
  • Pinch
    • Pinch Did Start
    • Pinch Moved
    • Pinch Did End
  • Pan
    • Pan Did Start
    • Pan Moved
    • Pan Did End

The point of this manager is to allow developers to use their time dealing with these touch events, instead of building logic to figure out when these touch events occur.

@asm09fsu asm09fsu added this to the 4.2 milestone May 19, 2016
@joeljfischer
Copy link
Contributor

Would it make sense to mirror iOS' own gesture and uiresponder architecture
for familiarity?
On Thu, May 19, 2016 at 5:33 PM Alex Muller notifications@github.com
wrote:

Bug Report

After dealing with using the onTouchEvent callback, I propose we design an
architecture for having a touch manager to give callbacks that allow
developers to know if certain touch events are happening. Currently the
following are thought of as important:

  • Taps
    • Single Tap
    • Double Tap
  • Pinch
    • Pinch Did Start
    • Pinch Moved
    • Pinch Did End
  • Pan
    • Pan Did Start
    • Pan Moved
    • Pan Did End

The point of this manager is to allow developers to use their time dealing
with these touch events, instead of building logic to figure out when these
touch events occur.


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#402

@asm09fsu
Copy link
Contributor Author

I considered that, however if we do something similar to how the gesture recognizer's work is there is still a catch all function for the respective gesture (tap, pinch, pan). I tried to think of it by completely extracting out the logic needed to determine what state you are currently in, and make it a bit more clean.

@asm09fsu asm09fsu mentioned this issue Jun 15, 2016
9 tasks
@joeljfischer
Copy link
Contributor

Fixed in #423

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

No branches or pull requests

2 participants