Hammer.js: Scroll isn't disable on drag events with iOS 9.3

Created on 29 Mar 2016  Â·  18Comments  Â·  Source: hammerjs/hammer.js

Hi everybody,

Scroll isn't disable on drag events with iOS 9.3.
I have compared with old ios version and all it's good !

Best,

Adrien

Patch Release bug iOS

Most helpful comment

We will do a new release with this and some other fixes some time this week i would think.

All 18 comments

Thank you for reporting this. Im already working on a patch for it. This is because of iOS incomplete implementation of touch-action we now need to not just check if its supported but if it supports none

Ok ! Thanks @arschmitz for your precisions

Hey guys. What is the best way to interim fix this?
(BTW: Keep up the great work!)

im not sure there is a simple fix working to get this fixed but been busy hope to get it done this weekend

Read last message ! https://bugs.webkit.org/show_bug.cgi?id=149854
I did not understand everything, but it may be a solution…

@adrienbecker thats unrelated to this but thank you

failed

@adrienbecker no worries that was the cause with some issues in some of my other libraries not totally off base or anything just not this particular issue. Thats for touch-action manipulation to work ( the only value supported by webkit ). In your case touch action none is the issue which is just not supported by webkit at all. We fake it when its not supported. Problem is we check for support of touch-action as a whole not the individual props since we never expected someone ( apple ) to implement just one value and not the rest.

I guess this can be checked with CSS.supports('touch-action', 'none');. But I don't have an iPhone to confirm that this will work. Do you @arschmitz? This will be an easy fix then. In touchAction.js is a condition to detect native support, we could easily enhance it.

https://github.com/hammerjs/hammer.js/pull/955 a PR, can anyony verify this fix?

@jtangelder I have an iPhone but I've avoided upgrading to 9.3 due to this... :/

Guess I need to take the leap?

@jtangelder @runspired i have one upgraded @runspired see comment on #955 though

@arschmitz Great ! It works fine :)

Can you update the npm package ?

We will do a new release with this and some other fixes some time this week i would think.

Any updates on the npm release?

Pull request #956 is always open

For anyone coming across this, a quick fix might be to call event.preventDefault() in your handler. This has worked for me for pan and pinch (haven’t tried others).

Was this page helpful?
0 / 5 - 0 ratings