Closed Bug 1123069 Opened 9 years ago Closed 6 years ago

[hamachi] Not possible to open link located on horizontal swipe carousel

Categories

(Web Compatibility :: Site Reports, defect, P5)

ARM
Gonk (Firefox OS)
defect

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: zrzut01, Unassigned)

References

()

Details

(Whiteboard: [contactready] [js] [country-all] [lib-swipe.js])

Attachments

(1 file)

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:35.0) Gecko/20100101 Firefox/35.0
Build ID: 20150108202552

Steps to reproduce:

1. Open mobile version of gsmarena.com (same on m.interia.pl)
2. Page should open on Home seciton with carousel with news on the bottom.
3. It is possible to go through 6 news on the carousel.


Actual results:

Not possible to open any of news from the horizontal carousel as shown on the yt video:

https://www.youtube.com/watch?v=pcUmfE_TvzY


Expected results:

It should be possible to open link which is located on horizontal swipe carousel.
Found on:

Alcatel One Touch Fire production (got from T-mobile Poland)
B2G version: 3.0.0.0-prerelease master
Platform version: 38.0a1
Build Identifier: 2015015010229
Git commit info: 2015-01-16 22:12:30 0f65b258
OS: All → Gonk (Firefox OS)
Hardware: All → ARM
Whiteboard: [hamachi]
Same on
Alcatel One Touch Fire production (got from T-mobile Poland)
B2G version: 3.0.0.0-prerelease master
Platform version: 38.0a1
Build Identifier: 20150118010206
Git commit info: 2015-01-16 22:12:30 0f65b258

By the way, if I remember well, that was also on B2G 2.2 and (I'm not sure about this) on 2.1
The issue exists from the time when old Browser was replaced by the new one. b2g 2.0 was the last version on which the scenario worked properly.
Flags: needinfo?(bugmail.mozilla)
I can repro on Hamachi but not Flame. I'll investigate, parking with me for now.
Assignee: nobody → bugmail.mozilla
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(bugmail.mozilla)
Summary: Not possible to open link located on horizontal swipe carousel → [hamachi] Not possible to open link located on horizontal swipe carousel
Whiteboard: [hamachi]
Seems like the webpage is doing a preventDefault on the first touchmove, which prevents the tap gesture from being detected. On hamachi the first touchmove coordinates are at the same point as the touchstart so it's technically not a "move". I haven't checked on Flame yet but if my memory serves I think on Flame we wouldn't be sending that touchmove at all, and so the webpage wouldn't preventDefault it, and the tap would get detected.
The website is using an old version of swipe.js [1], which has the attached touchstart and touchmove listeners. Note in particular that the code will detect a horizontal scroll even if deltaX and deltaY are both 0 on the first touchmove, which is exactly what is happening here. The latest version of swipe [2] appears to have the same problem. I think this is badness in the JS code; most JS libraries that attempt to do similar things ensure that they allow some slop for the finger to move before attempting to detect the direction. As it is code like this defeats the allowed slop in our APZ code and will make it much harder to tap things (if your finger wiggles even a tiny bit between touchdown and touchup it will misinterpret it).

[1] http://cdn2.gsmarena.com/w/m/js/swipe.js
[2] https://github.com/thebird/Swipe/blob/master/swipe.js#L302
Moving to Tech Evangelism since I think the website/libraries should be updated in this case.
Assignee: bugmail.mozilla → nobody
Component: Gaia::Browser → Mobile
Product: Firefox OS → Tech Evangelism
Seems this is well understood, marking contactready. But is this working better in other browsers?
Whiteboard: [contactready] [js] [country-all] [lib-swipe.js]
Same situation in Facebook app on hamachi. If you log in to Facebook, go to Friends section then on the top you will also have carousel with links and it also doesn't work.
Component: Mobile → Preinstalled B2G Apps
désolé, wrong move.
Component: Preinstalled B2G Apps → Mobile
Priority: -- → P5
The Firefox OS project is not active anymore.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
Product: Tech Evangelism → Web Compatibility
Component: Mobile → Site Reports
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: