Skip to content
This repository has been archived by the owner on Aug 10, 2022. It is now read-only.

Possibly better to use changedTouches than targetTouches #21

Closed
pp-koch opened this issue Apr 30, 2014 · 2 comments
Closed

Possibly better to use changedTouches than targetTouches #21

pp-koch opened this issue Apr 30, 2014 · 2 comments
Assignees
@gauntface
Milestone
v2

Comments

@pp-koch
Copy link

pp-koch commented Apr 30, 2014

The preference for targetTouches is not explained anywhere, and I think changedTouches is preferable because it will also work ontouchend, when touches and targetTouches are empty. This is especially important for scripts like drag and drop where the touchend event contains important information. As far as I know there is no drawback to using changedTouches as long as you're working with one touch at the time.
@gauntface
Copy link

I think it was your last point which made me opt for targetTouches - "as long as you are working with one touch at a time".

Looking at the touchmove and touchend using targetTouches and changedTouches, I can't see a difference from the last touchmove and the final touchend.

Testing with: http://jsbin.com/wezacuya/quiet

@PaulKinlan PaulKinlan added this to the 1.0 launch milestone May 9, 2014
@gauntface
Copy link

Will stick with targetTouches for now.

Happy to change if anyone disagrees.

cc @pp-koch

gauntface pushed a commit that referenced this issue Nov 15, 2017
# This is a combination of 27 commits.
3c205d7 
# This is the 1st commit message:

Adding start of Workbox v3 alpha docs

# This is the commit message #2:

Adding v3 alpha

# This is the commit message #3:

Adding configuration guide

# This is the commit message #4:

Adding workbox-precaching and fixing nested parameters

# This is the commit message #5:

Setting up namespaced docs on WF

# This is the commit message #6:

Adding ref and module docs for precaching

# This is the commit message #7:

Adding ref and module docs for precaching

# This is the commit message #8:

Routing Docs and module guide

# This is the commit message #9:

Routing Docs and module guide

# This is the commit message #10:

Adding workbox-cache-expiration guide and reference docs

# This is the commit message #11:

Adding routing docs

# This is the commit message #12:

Changing menu name

# This is the commit message #13:

Adding ability to use custom project yaml

# This is the commit message #14:

Adding workbox pages

# This is the commit message #15:

Getting it to work on devsite staging

# This is the commit message #16:

Adding some headings instead of sections but need to stage

# This is the commit message #17:

Moving to heading

# This is the commit message #18:

Adding heading support for navigation

# This is the commit message #19:

Adding contributor list

# This is the commit message #20:

Adding more content for docs

# This is the commit message #21:

Adding in company logos

# This is the commit message #22:

Adding in company logos

# This is the commit message #23:

Adding logos

# This is the commit message #24:

Adding some fixes for left nav

# This is the commit message #25:

Renaming strategies

# This is the commit message #26:

Fixng ref doc bug and add missing module pages

# This is the commit message #27:

Adding Guides etc
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@gauntface gauntface

Labels
None yet
Projects
None yet
Milestone
v2
Development

No branches or pull requests
3 participants
@PaulKinlan @gauntface @pp-koch