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

iPad scroll zoom issue #8595

Closed
RedAuburn opened this issue Jul 26, 2021 · 7 comments
Closed

iPad scroll zoom issue #8595

RedAuburn opened this issue Jul 26, 2021 · 7 comments
Assignees
Milestone

Comments

@RedAuburn
Copy link

RedAuburn commented Jul 26, 2021

I am unable to scroll-zoom when using an iPad with a dedicated mouse or magic keyboard trackpad. The issue happens on both the embedded version of iD on osm.org, and on http://ideditor.netlify.app/

Tried Firefox and Safari, it happened on both

Video of the issue:
https://www.youtube.com/watch?v=sKRmxKWuCIc

@mbrzakovic
Copy link
Collaborator

Hi @endim8!
Which version of os is your iPad using?

@RedAuburn
Copy link
Author

RedAuburn commented Jul 28, 2021

iPadOS 15 Beta 3, although it's been happening at least since 14.6 (I only got my iPad recently, so I haven't tried before 14.6)

@RedAuburn
Copy link
Author

I've sold my iPad so can't test this specific issue anymore, however I was having a similar issue on my MacBook that replaced it.
After looking at #5550 and then webcompat/web-bugs#73148, It seems that setting dom.event.wheel-deltaMode-lines.disabled to false in Firefox's about:config fixes it

@mbrzakovic mbrzakovic self-assigned this Aug 12, 2021
@mbrzakovic mbrzakovic added this to the 2.20.1 milestone Aug 13, 2021
@mbrzakovic
Copy link
Collaborator

For reference: #5492,
68be75a#diff-a1008158a41859a7de040d1a3db175073f92aeab5433ea6d1f958319993555aa
4f8f45e

Issue: delta y value for both 2 finger panning and mouse scroll on mac os mozila firefox was always integer and therefore mouse scroll was being treated as 2 finger panning mode.
Solution implemented: Remove 2 finger panning handling for mac os firefox as map zoom in/out is of higher importance.

@mbrzakovic
Copy link
Collaborator

@endim8 please double check this on https://ideditor.netlify.app/

@RedAuburn
Copy link
Author

@endim8 please double check this on https://ideditor.netlify.app/

Can confirm, it's fixed 😄 thank you!

@1ec5
Copy link
Collaborator

1ec5 commented Aug 21, 2021

The scrolling in Safari on iPad, seen in the original video, looks more like #7576 than #5550 or webcompat/web-bugs#73148.

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

3 participants