753201 - huffingtonpost.com/mobileweb article links are not tappable

Status: RESOLVED WORKSFORME

(Web Compatibility :: Site Reports, defect)

Description

[Chris Peterson [:cpeterson]]

STR:
1. Load http://www.huffingtonpost.com/mobileweb
2. Scroll down to the list of "TOP NEWS" articles
3. Tap a TOP NEWS article's title or picture

AR:
Nothing happens!

ER:
Tapping or clicking the article title or picture should load the article page.

These links work correctly on Chrome Beta, ICS stock browser, Gingerbread stock browser, Mac Chrome desktop browser, Mac Safari desktop browser, and Mac Firefox desktop browser.

This page seems to be using some JS tricks for link navigation, BUT I do not think this is a case of webkit-only code because the links work correctly with the Mac Firefox desktop browser.

Comment 1

[Kevin Brosnan [Ex-Mozilla]]

Sounds like bug 719017

Comment 3

[Kartikaya Gupta (email:[e-mail])]

When I try to click those links on Fennec, I get this error in logcat:

E/GeckoConsole(17069): [JavaScript Error: "ReferenceError: WebKitCSSMatrix is not defined" {file: "http://s.huffpost.com/assets/js.php?f=sprintf-0.7.js%2Cjquery.js%2Cjquery%2Fjquery.I18N.js%2Clocale%2Fdefault.js%2Chp_app.js%2Chuff.js%2Chp_track.js%2Chp_config.js%2Ccookiesmin.js%2Cfacebook.js%2Chp_util.js%2Chp_browser.js%2Csnproject.js%2C_tmp%2Fmodal.js%2C_tmp%2Fuser_poll.js%2C_tmp%2Fquick_snp.js%2C_tmp%2Fquick_login.js%2C_tmp%2Fquick_facebook_invite.js%2C_tmp%2Ffan_action.js%2C_tmp%2Fquick_subscribe_user.js%2C_tmp%2Fquick_signup.js%2C_tmp%2Fquick_fan.js%2C_tmp%2Fsay_it.js%2C_tmp%2Fquick_contribute.js%2C_tmp%2Flightbox_search.js%2C_tmp%2Femail_alerts.js%2Chuffconnect.js%2Cprovider.js%2Cmodal_window.js%2Cpopup_manager.js%2Cmobile%2Fauth.js%2Cmobileweb%2Fnews%2Fiscroll-min.js%2Cmobileweb%2Fmobile_metrics.js%2Cmobileweb%2Fnews%2Fnews.js&v79534" line: 4264}]

The page has a touch event handler which assumes webkit, and tries to do some webkit-ism. This blows up (but not before canceling the default action of the touch event, I believe) so we never trigger the click. On desktop this works fine because the touch event listeners are not run, we just run the click listener directly.

Moving to evangelism, they should stop giving us broken code.

Comment 4

[Kartikaya Gupta (email:[e-mail])]

Also, for future reference, the JS code in question looks something like this:

e.stopPropagation();that.scrolling=true;that.moved=false;that.distX=0;that.distY=0;that.setTransitionTime('0');if(that.options.momentum||that.options.snap){matrix=new WebKitCSSMatrix(window.getComputedStyle(that.element).webkitTransform);if(matrix.e!=that.x||matrix.f!=that.y){document.removeEventListener('webkitTransitionEnd',that,false);that.setPosition(matrix.e,matrix.f);that.moved=true;}}

Comment 6

[Aaron Train [:aaronmt]]

Apparently this is a Top-30 mobile site that needs some evangelism magic.

Comment 7

[Kev Needham [:kev]]

Will ping our contacts at Aol.

(In reply to Aaron Train [:aaronmt] from comment #6)
> Apparently this is a Top-30 mobile site that needs some evangelism magic.

Comment 8

[Jason Smith [:jsmith]]

(In reply to Kev [:kev] Needham from comment #7)
> Will ping our contacts at Aol.
> 
> (In reply to Aaron Train [:aaronmt] from comment #6)
> > Apparently this is a Top-30 mobile site that needs some evangelism magic.

Do you need a summarized report? We've got 4 evangelism bugs on the meta bug linked to this (might as well mention all 4 in one sweep).

Comment 9

[Kev Needham [:kev]]

If you could email me all the bug numbers, I'll amend the request.

k

Comment 10

[Jason Smith [:jsmith]]

(In reply to Kev [:kev] Needham from comment #9)
> If you could email me all the bug numbers, I'll amend the request.
> 
> k

Looks like you figured it out reading the email (as it's the correct numbers) of bug 749744, bug 752033, bug 753201, and bug 757151.

Comment 11

[Aaron Train [:aaronmt]]

Works for me now (/mobileweb)

Their desktop site is still to this day serving corrupted content to the Fennec UA.