manubb / Leaflet.PixiOverlay

Bring Pixi.js power to Leaflet maps
MIT License
463 stars 85 forks source link

Doesn't work click event on map on Mobile #16

Closed emehmet closed 6 years ago

emehmet commented 6 years ago

Hi, Doesn't work click event on map on Mobile. How can I use?

manubb commented 6 years ago

What is your device? os? browser? Are you experiencing this on any homepage demo?

emehmet commented 6 years ago

Excuse me. I tested your examples. https://manubb.github.io/Leaflet.PixiOverlay/leaflet-quickstart.html. And My Phone is LG G4 (android). Browser is Chrome V:65

manubb commented 6 years ago

The other examples with interactivity are: Largest US cities. French presidential 2017 election results: first round. Can you tell me if the problem also occurs on them? (Unfortunately, i have no android device to test on.)

emehmet commented 6 years ago

Yes. I get same result for all examples too.

manubb commented 6 years ago

I have pushed a fix for the demos (commits e0c93b6d7b1b7d8fd16addbf96b4b19ec885fd0f and 6658acf6e923768afffd934c94f70899dd305607. Last will be included in v1.5.0). Can you check and tell me if the problem is solved? Thanks.

emehmet commented 6 years ago

I checked very basic demo, Largest US cities (1000 animated markers), French cities (36700 animated markers) and French presidential 2017 election results. All demos worked properly except (very basic demo) on Android phone version. Sometimes very basic demo works. Sometimes not. I tried to understand how work. I think there is a difference between clicking on the map and clicking on the marker. When I tangent the marker on the screen, the popup opens. But when I click directly on the marker, it does not always pop up. The same result applies to polygons. Thanks for your great job.

manubb commented 6 years ago

Glad to read that almost of demos are fixed! Can you recheck the very basic demo, please? (I needed some times to fix this one. I fell into the hell of leaflet events and pixi.js events...)

emehmet commented 6 years ago

Okay. Everything looks good. Demos is working properly on Android. Thanks for your great job and great project.

manubb commented 6 years ago

Thanks for reporting!