r97381 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:< r97380‎ | r97381 | r97382 >
Date:18:01, 17 September 2011
Author:krinkle
Status:ok (Comments)
Tags:
Comment:
Adding padding:0px to '.catlinks ul' (Follow-up r92054). Browsers sometimes have a default padding-left: 40px; which looks ugly if the <ul> is diplayed inline with a label in front of it. Before: http://i.imgur.com/JWJvp.png ; After: http://i.imgur.com/Vo1nN.png
Modified paths:
  • /trunk/phase3/skins/common/shared.css (modified) (history)

Diff [purge]

Index: trunk/phase3/skins/common/shared.css
——@@ -286,6 +286,7 @@
287287 .catlinks ul {
288288 display: inline;
289289 margin: 0px;
 290+ padding: 0px;
290291 list-style: none;
291292 list-style-type: none;
292293 list-style-image: none;

Follow-up revisions

RevisionCommit summaryAuthorDate
r974141.18: MFT r96775, r97362, r97365, r97381catrope11:36, 18 September 2011
r99935Add 'padding: 0;' for '.catlinks ul' in commonPrint as well...krinkle23:51, 15 October 2011

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r92054Render category links as an HTML list. Bug 12261. Based on patch by Thana & B...diebuche10:41, 13 July 2011

Comments

#Comment by Fomafix (talk | contribs)   04:12, 8 October 2011

padding:0 is also missing in phase3/skins/common/commonPrint.css from r96775.

Status & tagging log