766035 - PhotoSwipe.JS - Mobile Roadie PJs Coffee - Selecting to View Photos Never Finishes Loading on FF for Android

Status: RESOLVED INVALID

(Web Compatibility :: Site Reports, defect)

Description

[Jason Smith [:jsmith]]

Steps:

1. Go to http://mobileroadie.com/m/pjs-coffee
2. Select PJ's Coffee Pics

Expected:

A thumbnail listing of photos should show in an app-like format.

Actual:

The page continuously sits spinning the loading dial, but nothing ends up loading.

Additional Notes:

Issues does not reproduce on the stock browser or desktop firefox.

Comment 1

[Aaron Train [:aaronmt]]

I can reproduce this changing my user-agent on desktop to mobile Firefox. I get a client-side error:

Uncaught TypeError: Cannot call method 'toString' of null

h.PhotoSwipeClass.d.initialize
l
a.createInstance
a.attach
e.jQuery.fn.photoSwipe
(anonymous function)
f.Callbacks.n jquery-1.7.1.min.js:2
f.Callbacks.o.add jquery-1.7.1.min.js:2
e.fn.e.ready jquery-1.7.1.min.js:2
(anonymous function)
(anonymous function)
e.extend.globalEval jquery-1.7.1.min.js:2
e.extend.globalEval jquery-1.7.1.min.js:2
bp jquery-1.7.1.min.js:2
e.extend.each jquery-1.7.1.min.js:2
f.fn.extend.domManip jquery-1.7.1.min.js:4
f.fn.extend.append jquery-1.7.1.min.js:3
f.each.f.fn.(anonymous function) jquery-1.7.1.min.js:4
a.mobile.loadPage.a.mobile.allowCrossDomainPages.l.isSameDomain.a.ajax.success jquery.mobile-1.1.0.min.js:58
f.Callbacks.n jquery-1.7.1.min.js:2
f.Callbacks.o.fireWith jquery-1.7.1.min.js:2
w jquery-1.7.1.min.js:4
f.support.ajax.f.ajaxTransport.send.d


PhotoSwipe.JS which is from http://www.codecomputerlove.com/

I'll send them an email.

Comment 2

[Aaron Train [:aaronmt]]

PhotoSwipe.JS library updated, Mobile Roadie needs to use the new library

Comment 3

[Aaron Train [:aaronmt]]

Site URL changed to: http://m.zipscene.com/pjs-coffee/, and this works now.

Comment 4

[Adrian Tamas (:AdrianT)]

The issue is still reproducible at both http://m.zipscene.com/pjs-coffee/ and http://mobileroadie.com/m/pjs-coffee on Nightly 18.0a1 2012-08-28, Aurora 16.0a2 2012-08-27 and Firefox Mobile 15 RC using a Motorola Droid 3 running Android 2.3.4 and a Nexus Galaxy R running Android 2.3.4. Also the issue is still reproducible when changing the user agent on Desktop Firefox to the Android User Agent but is not reproducible if the link is loaded directly

Comment 5

[Lawrence Mandel [:lmandel] (use needinfo)]

I can also reproduce the issue with Nightly 18.0a1 2012-08-28 on a Galaxy S2 with ICS 4.0.4.

STR:
1. Open http://m.zipscene.com/pjs-coffee/
2. Click on Photos
3. Click on the library that is listed (Simply the Best)

Expected results:
Photo library loads.

Actual results:
I see a spinner that just continues to spin.

Aaron - On what device did you verify the fix?

Comment 6

[Hallvord R. M. Steen [:hallvors]]

Script is a bit careless when trying to extract the Android version. 

/zmp-m/js/code.photoswipe.jquery-2.1.6.min.js contains:

if(a.Browser.android&&e.navigator.userAgent.match(/Android (\d+.\d+)/).toString().replace(/^.*\,/,"")>=2.1)this.isBackEventSupported=!0;if(!this.isBackEventSupported)this.isBackEventSupported=a.objectHasProperty(e,"onhashchange");

The code throws because without any version number after "Android" in the UA, match() returns null.

Comment 7

[Karl Dubost💡 :karlcow]

Trying to contact through twitter.
http://twitter.com/MozWebCompat/status/429137012059164672

Comment 8

[Karl Dubost💡 :karlcow]

It seems to be working on the Aurora 32 I have on the Android device.
I will close it as FIXED.

Comment 9

[Hallvord R. M. Steen [:hallvors]]

Still an issue (with the normal Fx Android UA string)

Comment 10

[Karl Dubost💡 :karlcow]

http://m.zipscene.com/pjs-coffee/
Doesn't exist anymore.