603194 - Wikipedia's mobile site is unreadable

Status: VERIFIED FIXED

(Tech Evangelism Graveyard :: English Other, defect)

Description

[Benjamin Stover (:stechz)]

It has small text with a full-size viewport with columns that are the width of the viewport.

Comment 1

[Matt Brubeck (:mbrubeck)]

I can reproduce this on Android, but not on desktop Linux.  It looks like Wikipedia's mobile site sends different markup to Fennec on Android.  On desktop I see this:

<meta name = "viewport" content = "width=device-width; initial-scale=1.0; maximum-scale=1.0; user-scalable=0;" />

But on Android there is no meta[name="viewport"] element.

Comment 2

[Aakash Desai [:aakashd]]

This should block 2.0 since it's a default search option

Comment 3

[Benjamin Stover (:stechz)]

(In reply to comment #2)
> This should block 2.0 since it's a default search option

If Matt is right, there's nothing we can really do about this except get in touch with Wikipedia or remove it as a search option.

Comment 4

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

My contact at Wikipedia is forwarding this along to the right people.  Will report back when I have an update.

Comment 5

[Chris Lawson (gone)]

Shouldn't this stay in TE, then, since it's really a Wikipedia bug and not a Fennec bug?

Comment 6

[Mark Finkle (:mfinkle) (use needinfo?)]

(In reply to comment #5)
> Shouldn't this stay in TE, then, since it's really a Wikipedia bug and not a
> Fennec bug?

We want this to be fixed and if we can do something to fix it, we should. I'm afraid TE is a black hole of FAIL.

Matt - Is this a "Android 2.x" UA issue?

Comment 7

[Chris Lawson (gone)]

(In reply to comment #6)
> (In reply to comment #5)
> > Shouldn't this stay in TE, then, since it's really a Wikipedia bug and not a
> > Fennec bug?
> 
> We want this to be fixed and if we can do something to fix it, we should. I'm
> afraid TE is a black hole of FAIL.

Not if someone actually owns the bug and is willing to work on it. :) The biggest problem TE has is lack of resources to evangelise. If you -- or others on the Fennec team -- are willing to be point people on this, it'll get fixed, barring total obstinacy on the part of Wikipedia.

cl

Comment 8

[Mark Finkle (:mfinkle) (use needinfo?)]

(In reply to comment #7)
> it'll get fixed, barring total obstinacy on the part of Wikipedia.

That's my concern

Comment 9

[Chris Lawson (gone)]

(In reply to comment #8)
> (In reply to comment #7)
> > it'll get fixed, barring total obstinacy on the part of Wikipedia.
> 
> That's my concern

With all due respect, I don't think Wikipedia's inclination to fix this or not has anything to do with what component of Bugzilla the bug is filed in. :-p

cl

Comment 10

[Mark Finkle (:mfinkle) (use needinfo?)]

(In reply to comment #9)
> (In reply to comment #8)
> > (In reply to comment #7)
> > > it'll get fixed, barring total obstinacy on the part of Wikipedia.
> > 
> > That's my concern
> 
> With all due respect, I don't think Wikipedia's inclination to fix this or not
> has anything to do with what component of Bugzilla the bug is filed in. :-p

Indeed, and you can create a clone of this in TE. It doesn't remove the fact we still may need to do something in product to fix this. Putting this bug in a component that core developers do not track will remove it from the "radar", which I don't want.

Comment 12

[Brad Lassey [:blassey] (use needinfo?)]

Attachment: screen shot

this is what I see

Comment 13

[Brad Lassey [:blassey] (use needinfo?)]

in content.js viewportScale is NaN, which is why we're not laying this out correctly

Comment 14

[Brad Lassey [:blassey] (use needinfo?)]

m.wikipedia.com has this for a viewport meta tag:
<meta name = "viewport" content = "width=device-width; initial-scale=1.0; maximum-scale=1.0; user-scalable=0;" />

howver, windowUtils.getDocumentMetadata("viewport-initial-scale") is returning an empty string. Something is broken here.

Comment 15

[Matt Brubeck (:mbrubeck)]

(In reply to comment #14)
> m.wikipedia.com has this for a viewport meta tag:
> <meta name = "viewport" content = "width=device-width; initial-scale=1.0;
> maximum-scale=1.0; user-scalable=0;" />
> 
> howver, windowUtils.getDocumentMetadata("viewport-initial-scale") is returning
> an empty string. Something is broken here.

That might be fixed by bug 572696, which was just pushed a few hours ago.

Comment 16

[Brad Lassey [:blassey] (use needinfo?)]

(In reply to comment #15)
> (In reply to comment #14)
> > m.wikipedia.com has this for a viewport meta tag:
> > <meta name = "viewport" content = "width=device-width; initial-scale=1.0;
> > maximum-scale=1.0; user-scalable=0;" />
> > 
> > howver, windowUtils.getDocumentMetadata("viewport-initial-scale") is returning
> > an empty string. Something is broken here.
> 
> That might be fixed by bug 572696, which was just pushed a few hours ago.

turns out wikipedia isn't giving us the viewport meta tag on android. It gives it to us on desktop and we handle it correctly, so this sounds like an evangelism bug now.

Comment 17

[Doug Turner (:dougt)]

Let's remove wikipedia from the search list until nicer content is available for us.

Comment 18

[Mark Finkle (:mfinkle) (use needinfo?)]

Kev - Would this fall under your realm of influence? Can we reach out to the folks at Wikipedia to get send Fennec consistent content?

Comment 19

[Mark Finkle (:mfinkle) (use needinfo?)]

We have choices for content:
* Full web site (http://en.wikipedia.org)
* Plain mobile web site (http://mobile.wikipedia.org)
* Enhanced mobile web site (http://en.m.wikipedia.org)

I think we want the enhanced mobile web site as a default for Fennec. However, we still have a problem:

If you visit http://en.m.wikipedia.org in Fennec on Maemo or Desktop, you'll get a page that has a <meta name="viewport"> element and uses /stylesheets/default.css

If you visit http://en.m.wikipedia.org in Fennec on Android, you'll get the same content, but there is no <meta name="viewport"> tag and /stylesheets/android.css is used.

The missing viewport meta tag causes the zooming problem. The android.css has some webkit specific CSS in it.

If Wikipedia could always send the <meta name="viewport"> use default.css, the content would look optimum in Fennec.

Comment 20

[Aakash Desai [:aakashd]]

I'm talking to the folks at wikipedia and they're going to url sniff for fennec and show us their iphone site; will update when it's done.

Comment 21

[Matt Brubeck (:mbrubeck)]

This was fixed by Wikipedia.

Comment 22

[Aakash Desai [:aakashd]]

Yep, going over to en.m.wikipedia.org shows the iphone version on fennec nightlies now.