New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 670942 link

Starred by 4 users

Issue metadata

Status: WontFix
Owner:
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug-Regression



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 description

Chrome 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.
 

Comment 1 Deleted

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).


Comment 3 by ajha@chromium.org, Dec 5 2016

Labels: -Type-Bug -Pri-3 M-55 OS-Android Pri-1 Type-Bug-Regression
Cc: aelias@chromium.org dtapu...@chromium.org candr...@chromium.org amineer@chromium.org
Labels: hasbisect-per-revision
Owner: mustaq@chromium.org
Status: Assigned (was: Unconfirmed)
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


Components: Blink>Input
At first glance, looks like OpenLayers Map Viewer is incorrectly handling pointer events detection: https://github.com/openlayers/ol2
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.
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
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.
Labels: -Pri-1 Pri-3
Status: ExternalDependency (was: Assigned)
Thanks very much for looking at this and tracking it down so much!
matthew@ are you able to patch the ol2 software as I indicated? I've done it locally and what I indicated works correctly.
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.
Status: WontFix (was: ExternalDependency)
Closing issue as this issue is now fixed and wasn't an issue with chromium.
 Issue 672770  has been merged into this issue.
dtapuska@ & Blink crew, note that the bug that was just merged in references another JS library (leaflet js library).
Summary: Open Layers v2 maps do not drap in Chrome Beta (55) on Android (was: Dragging map does nothing in Chrome Beta (55) on Android)

Comment 18 by i...@mazemap.no, 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).
Labels: Hotlist-Input-Dev
Drag no longer works in OpenLayers 3. We fixed it by restoring WebView to factory settings.

Sign in to add a comment