Issue metadata
Sign in to add a comment
|
Open Layers v2 maps do not drap in Chrome Beta (55) on Android
Reported by
matt...@mysociety.org,
Dec 3 2016
|
||||||||||||||||||||||
Issue descriptionChrome Version : 55.0.2883.77 on Android 5.1 URLs (if applicable) : https://www.fixmystreet.com/around?zoom=3&lat=51.48583&lon=-3.21572&layers=B Other browsers tested: Add OK or FAIL, along with the version, after other browsers where you have tested this issue: Safari: OK, 10 Firefox: OK, 49 IE: OK, Edge Chrome 54 on Android: OK Chrome 55 Beta on Mac: OK What steps will reproduce the problem? (1) Visit above web page on Android phone (2) Try and drag map (3) Nothing happens What is the expected result? The map should be a slippy map, draggable by the user. What happens instead? Nothing :( The manual pan/zoom buttons work fine, so the map code is working okay, except for dragging itself.
,
Dec 3 2016
I have made a cut down version that exhibits the same issue using an OpenLayers demo as a starter point: http://dracos.co.uk/temp/2016-12-03-chrome-drag/ Double-click to zoom is fine, but map does not drag in Chrome 55 on Android (Chrome 55 on desktop is fine).
,
Dec 5 2016
,
Dec 8 2016
Able to repro this issue on Nexus 6 / MOB31O Good build: 55.0.2875.4 Bad build: 55.0.2876.0 Full CL list: https://chromium.googlesource.com/chromium/src/+log/55.0.2875.0..55.0.2876.0?pretty=fuller&n=10000 Bisect script pointed to https://chromium.googlesource.com/chromium/src/+/902a3d6303dfbfd49a24173777afcb4b70234a17 https://codereview.chromium.org/2375493005
,
Dec 9 2016
At first glance, looks like OpenLayers Map Viewer is incorrectly handling pointer events detection: https://github.com/openlayers/ol2
,
Dec 9 2016
Adding a "touch-action: none" to map_box style seems to make the map pan for me on a chrome os touch device. Seems like a site issue.
,
Dec 9 2016
err; still need to investigate it. It doesn't seem to be working for me now on my test here: http://output.jsbin.com/mareweb
,
Dec 9 2016
The problem is two fold 1) No touch-action 2) touch events are added as well I've updated my jsbin to address these problems. Note that this code will probably only work in chrome but it is demonstrating that it is completely a site/framework issue. I've tried on Edge on a Windows phone and can confirm that the maps in comment #1 and #2 do not pan. yet the map in #7 does.
,
Dec 9 2016
I opened issue https://github.com/openlayers/ol2/issues/1510
,
Dec 9 2016
,
Dec 9 2016
Thanks very much for looking at this and tracking it down so much!
,
Dec 9 2016
matthew@ are you able to patch the ol2 software as I indicated? I've done it locally and what I indicated works correctly.
,
Dec 9 2016
Hi, your changes (but with the touch-action: none on olMap rather than olMapViewport) are now live on www.fixmystreet.com and it works here in Chrome 55 on Android, thank you. (Aside: though turns out our OL’s style.css isn’t versioned, which meant I had to clear my phone’s cache, will have to fix that!) I’ve commented on the GitHub ticket also and submitted the changes as a PR.
,
Dec 9 2016
Closing issue as this issue is now fixed and wasn't an issue with chromium.
,
Dec 9 2016
Issue 672770 has been merged into this issue.
,
Dec 9 2016
dtapuska@ & Blink crew, note that the bug that was just merged in references another JS library (leaflet js library).
,
Dec 9 2016
,
Dec 12 2016
Leaflet maintainer here. We're tracking this bug as https://github.com/Leaflet/Leaflet/issues/5180. It's going to take a while to get through all the casuistic, specially since we were making a special case for Blink on Win10 and touchscreen dblclicks (see https://github.com/Leaflet/Leaflet/issues/4127).
,
Jan 5 2017
,
Jan 20 2017
Drag no longer works in OpenLayers 3. We fixed it by restoring WebView to factory settings. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 Deleted