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.

Sign up for GitHub

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

Jump to bottom

Dragging path is not working in Chrome 55 with latest version of core Leaflet (1.0.3) #29

Open
peppierre opened this issue Feb 1, 2017 · 4 comments · May be fixed by #30
Open

Dragging path is not working in Chrome 55 with latest version of core Leaflet (1.0.3) #29

peppierre opened this issue Feb 1, 2017 · 4 comments · May be fixed by #30

Comments

@peppierre
Copy link

How to reproduce

Leaflet version I'm using: 1.0.3
Browser (with version) I'm using: Chrome55
OS/Platform (with version) I'm using: Android
Leaflet.Path.Transform version: 1.0.2
Leaflet.Path.Drag version: 1.0.6

What behaviour I'm expecting and which behaviour I'm seeing

Note that both Transform and Drag plugin (in this order) is referred.
Prior to Chrome 55 and using core Leaflet v1.0.2, plugin's _onDragStart() handler is called when a path is started to be dragged, as it is expected. In Chrome55, dragging paths doesn't work due to Chrome's newly introduced pointer events, I guess.
Unfortunately, updating to v1.0.3 of core did not fix this behavior in Chrome 55, dragging of path still doesn't work because event handler above is simply not called anymore.
@IvanSanchez
Copy link

If this needs any changes upstream, please leave comments at Leaflet/Leaflet#5298

@w8r
Copy link
Owner

w8r commented Feb 2, 2017 •
edited
Loading

@peppierre, it seems to be Android-specific? Cause I cannot reproduce it.

@IvanSanchez
Copy link

Yeah, I can reproduce it on Android. Seems that using a mouse pointer will make the browser fire both mousedown and pointerdown events, but using a touchscreen will fire only pointerdowns, no touchdowns.

@IvanSanchez IvanSanchez linked a pull request Feb 2, 2017 that will close this issue
Open
@Damorck
Copy link

Damorck commented Apr 16, 2017

Same issue here. So this is possible to fix? And is it working on Android with Leaflet 0.7?

@alexshk alexshk mentioned this issue Jun 11, 2017
Closed
2 tasks
lederer added a commit to azavea/pwd-waterworks-revealed that referenced this issue Jan 29, 2018
@lederer
Prototype: Make location marker draggable
4aeddc2 
This functionality is for the prototype only, to let the user simulate
precise location.

Note it doesn't not work in Android Chrome 55+:
w8r/Leaflet.Path.Drag#29
Leaflet/Leaflet#5556
https://github.com/alexshk/Leaflet/commit/2e55b7b796419832f07249b0cb651a3ca653f743
facebook/react#11817
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Blind attempt to fix Chrome>55 touchscreen behaviour
4 participants
@w8r @IvanSanchez @peppierre @Damorck