Page MenuHomePhabricator

Inline Styles: Portal pages need to be redesigned for mobile
Closed, ResolvedPublic

Description

Totally broken since we impose fixed width.

http://en.m.wikipedia.org/wiki/Portal:Xbox


Version: unspecified
Severity: major

Details

Reference
bz30887

Event Timeline

bzimport raised the priority of this task from to Low.Nov 21 2014, 11:55 PM
bzimport set Reference to bz30887.
bzimport added a subscriber: Unknown Object (MLST).

See bug 32123 for a bunch of info about fixing these portal pages; some have been cleaned up, others need to be done still.

It's better as now the table is horizontally scrollable but still messy.

Several problems:

  • Section toggling is not enabled. For some reason content of a section appears as a div outside the .content_block div rather than inside it
  • There seems to be a need for a PortalTemplate to take away the need for inline styling

So I've cleaned up lots of portal pages [1,2,3]

I think next step is to get a list of all pages which have known bad styles
To do this we should download one of the latest dumps [4] and grep them for the following known patterns:
float:left; width:60%
float:left; width:51%;
float:right; width:48%
and then edit those articles so that they use classes instead.

Gabrielle Wicke has several node grepping tools [4] that will help us do this
I'll report back with a list of bad pages so we can clean these up.

[1] http://en.wikipedia.org/w/index.php?title=Wikipedia:WikiProject_Microsoft&diff=prev&oldid=495628088
[2] http://en.wikipedia.org/w/index.php?title=Portal:Xbox&diff=prev&oldid=495624606
[3] http://en.wikipedia.org/w/index.php?title=Portal:Missouri&diff=prev&oldid=495624262
[4] https://gerrit.wikimedia.org/r/gitweb?p=mediawiki/extensions/VisualEditor.git;a=tree;f=tests/parser;h=daa09af44d1aaebbe3a5aa3d4a1b77b8e2531be9;hb=HEAD

philinje wrote:

Nice, but hard work. I wonder if we should stir up some community help, as we will shortly do with the default switchover of the sister projects?

Here is a list of effected pages on English Wikipedia:
http://www.mediawiki.org/wiki/List_of_Problematic_portal_pages_with_two_column_layouts

Please help clean these up to close this bug.

Another example of a page with an inline style problem:
http://en.m.wikipedia.org/w/index.php?title=New_Haven_Line

$("[style]").removeAttr("style") fixes it
Haven't got time right now to explore this further

All portal pages are now fixed via a grease monkey script
https://gist.github.com/3252773

  • Bug 22187 has been marked as a duplicate of this bug. ***

-dependency (which is marked duplicate)