Last modified: 2007-10-04 21:44:13 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T11718, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 9718 - Undo r11305 changes to main.css
Undo r11305 changes to main.css
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Interface (Other open bugs)
unspecified
All other
: Normal trivial (vote)
: ---
Assigned To: Nobody - You can work on this!
http://en.wikipedia.org/skins-1.5/mon...
: easy, patch, patch-reviewed
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2007-04-27 00:29 UTC by Splarka
Modified: 2007-10-04 21:44 UTC (History)
2 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Removes the offending css items (689 bytes, patch)
2007-10-04 21:36 UTC, Robert Leverington
Details

Description Splarka 2007-04-27 00:29:47 UTC
r11305 introduced some new CSS in an apparent redesign of the navigation
portlet, which uses the id definition #p-navigation in main.css. However, due to
the advent of MediaWiki:Sidebar, this only affects the top portlet (and only if
the first line is "* Navigation" (and possibly only in English?), as new sidebar
portlets are given an ID to match the portlet heading.

Part of the css defines "#p-navigation a { display: block; }" which gives the
top sidebar portlet links a slightly different spacing on some browsers (example
http://test.wikipedia.org/wiki/Image:Portletfunky.gif ).

It also seems, that #p-navigation li are no longer ever given class="active", so
probably the whole section in main.css under "** the navigation portlet" should
be removed as unneeded.
Comment 1 Robert Leverington 2007-10-04 21:36:26 UTC
Created attachment 4215 [details]
Removes the offending css items

This patch removes the block of #p-navigation css items, it doesn't appear to make the page look different and is broken regardless due to its implementation. If this really does change things for the better (which I cannot see and has never caused any complaints) there is no way to correctly implement it without changes to the classes in the skin as there are many .portal 's.
Comment 2 Daniel Cannon (AmiDaniel) 2007-10-04 21:43:22 UTC
Committed as r26419.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links