Page MenuHomePhabricator

Add a class attribute to the table on Special:Allpages
Closed, ResolvedPublic

Description

Author: kovacsur

Description:
The lines in the list of sections on Special:Allpages currently consist of three separate td elements, each containing a link to the same url. The cellspacing/cellpadding is not defined for the table containing the sections, causing the underlining to break, creating a look that is probably confusing (the links appear to point to the articles on each side, not to a page containing a list of pages from article A to article B).

A quick solution would be to add a class attribute to the table in question, so we could for instance add

table.allpageslist {border-spacing: 0px;}
table.allpageslist td {padding: 0}

to the site CSS and thus remove the unnecessary spacing, without risking the possibility of breaking other projects.

I'll attach a patch that does this.


Version: 1.6.x
Severity: trivial
URL: https://hu.wikipedia.org/wiki/Special:Allpages

Details

Reference
bz5565

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 9:12 PM
bzimport set Reference to bz5565.
bzimport added a subscriber: Unknown Object (MLST).

kovacsur wrote:

Patch to add a class attribute to the table on [[Special:Allpages]]

Attached:

kovacsur wrote:

Changing severity to "trivial", could someone please commit this to head? There is
not much to review about this one.

robchur wrote:

Added in SVN trunk, r13942.