Page MenuHomePhabricator

Left sidebar has extra separator section on many sites
Closed, ResolvedPublic

Description

From https://secure.wikimedia.org/wikipedia/en/wiki/Wikipedia:Village_pump_%28technical%29#Classic_skin_sidebar

FF7 is claimed as well. Reedy to post screenshot.


Version: 1.18.x
Severity: normal

Details

Reference
bz31436

Event Timeline

bzimport raised the priority of this task from to Normal.Nov 21 2014, 11:51 PM
bzimport added a project: MediaWiki-Interface.
bzimport set Reference to bz31436.
bzimport added a subscriber: Unknown Object (MLST).
Reedy added a comment.Oct 6 2011, 5:06 PM

Created attachment 9175
Screenshot of issue

IE 8.0.7601.17514

Attached:

brion added a comment.Oct 6 2011, 5:55 PM

I don't see excessive space there, but I do see an extra separator and an empty section. Is that the issue?

brion added a comment.Oct 6 2011, 5:58 PM

Ok yes, it's the separator. Removing the "on IE7 and IE8" and clarifying summary.

I can see this on en, fr, es, da.wikipedia.org so it seems pretty common.

TheDJ added a comment.Oct 9 2011, 3:22 PM

This is caused by the collection extension adding elements to the page that the Standard skin doesn't know how to handle.

Array
(

[navigation] => Array
    (
        [0] => Array
            (
                [text] => Main page
                [href] => /wiki/index.php/Main_Page
                [id] => n-mainpage-description
                [active] => 
            )

        [1] => Array
            (
                [text] => Community portal
                [href] => /wiki/index.php/Sandbox:Community_portal
                [id] => n-portal
                [active] => 
            )

        [2] => Array
            (
                [text] => Current events
                [href] => /wiki/index.php/Sandbox:Current_events
                [id] => n-currentevents
                [active] => 
            )

        [3] => Array
            (
                [text] => Recent changes
                [href] => /wiki/index.php/Special:RecentChanges
                [id] => n-recentchanges
                [active] => 
            )

        [4] => Array
            (
                [text] => Random page
                [href] => /wiki/index.php/Special:Random
                [id] => n-randompage
                [active] => 
            )

        [5] => Array
            (
                [text] => Help
                [href] => /wiki/index.php/Help:Contents
                [id] => n-help
                [active] => 
            )

    )

[SEARCH] => Array
    (
    )

[TOOLBOX] => Array
    (
    )

[LANGUAGES] => Array
    (
    )

[coll-print_export] => <ul id="collectionPortletList"><li id="coll-create_a_book"><a href="/wiki/index.php?title=Special:Book&amp;bookcmd=book_creator&amp;referer=Testje%27+bagger" title="Create a book or page collection" rel="nofollow">Create a book</a></li><li id="coll-download-as-rl"><a href="/wiki/index.php?title=Special:Book&amp;bookcmd=render_article&amp;arttitle=Testje%27+bagger&amp;oldid=474&amp;writer=rl" title="Download a PDF version of this wiki page" rel="nofollow">Download as PDF</a></li><li id="t-print"><a href="/wiki/index.php?title=Testje%27_bagger&amp;debug=true&amp;printable=yes" title="Printable version of this page [p]" accesskey="p">Printable version</a></li></ul>

)

TheDJ added a comment.Oct 9 2011, 3:36 PM

Fixed in r99353

tagging bugs for Marcus to look at