suggested change to "swipe" recognizer

[annam]

Hello,

after upgrading to hammer v2 we noticed some inconsistencies in swipe detection:

for swipe to be triggered in v2 the velocity and direction are calculated from the last interval. so if you swipe but keep your finger on screen for a few milliseconds, velocity drops and direction may even be 'none'. this wasn't the case in v1, I haven't checked the code but the swipe listener was definitely more sensitive.

we've noticed that "swipe" is commonly (expected to be) triggered even after holding the finger down for some time after a swipe interaction, and this seems to be missed in v2.

we tried to override the velocity issue by using a negative velocity value, which unfortunately forces the swipe detection to disregard velocity all together but it's something that we can live with.

however to fix the direction "none" issue (or even wrong dimension if you slightly move your finger in any other direction while holding) we had to override the attrTest and emit functions of the swipe recognizer to use the offsetDirection value instead of the direction one. unfortunately this means we'll miss any updates to the swipe recognizer unless we compare and add them to our own overrides. it would be great if you cared to integrate this into the master hammer.js!

tnx!

[runspired]

+1

[jtangelder]

Could you create a PR for these changes?

[tosca]

I am so sorry to bother you, but you are the experts and I have been
working on this for 3 weeks. Before I 'punt', I would like to know if
hammer.js versoin 1/**works with multiple carousels on multiple bootstrap
nav-tabs pages. I can get multiple carousels to swipe independently on one
single page, but not on a one page tab container.
* super simple carousel
* animation between panes happens with css transitions
*/
function Carousel(element)
{ etc...

http://rickmaxwell.apphb.com/#!work

I do have hammer working on my website build, but when tabbing to another
"WORK" catagory, the images are images render 100px wide across the
screen... BUT work perfectly when you resize the browser window,
(naturally, you cannot resize phones and tablets, but if you swipe enough
times, the single img fills the screen.

Oddly, the first carousel in the WORKS catagory works perfectly, but the
tab-click to the second catagory, returns a "pane" element that is always
100px wide, initially, until you resize the screen.

Does hammer.js ver 1, support multiple carousels in separate tab-pane
containers?

I am eternally grateful.

Thank you and njoy!

Tosca Ragnini
tosca.ragnini@gmail.com
630.248.1990

http://www.toscatech.net%20 www.toscatech.net

On Tue, Aug 12, 2014 at 4:55 AM, Jorik Tangelder notifications@github.com
wrote:

Could you create a PR for these changes?

—
Reply to this email directly or view it on GitHub
#640 (comment).

[annam]

Sure i'll create a PR for this next week!

[bmesing]

+1
The current implementation make swiping almost unusable: The velocity is calculated only for a certain last interval, i.e. if you slow down at the very end of your gesture (as one often does especially with a mouse), swiping will not work.

[ghost]

+1

[src-code]

+1 Glad to see this addressed in PR #669, it makes it difficult to recognize aborted pans/swipes as well (eg, user swipes a carousel left, but then starts to swipe right before releasing... normally you could compare direction and offsetDirection, but a momentary hesitation causes direction == none)

[akruis]

I just tried PR #669 and get much better results on iOS.

• 1 for this PR.

p.s. and many many thanks for providing hammer.

[anitapillai]

The current implementation make swiping almost unusable on Desktops. PR #669 fixes the issue

[runspired]

Closing in favor of #806