MediaWiki r97381 - Code Review

Repository:	MediaWiki
Revision:	< r97380‎ \| r97381 (on ViewVC)‎ \| 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)

Diff [purge]

Index: trunk/phase3/skins/common/shared.css
—	—	@@ -286,6 +286,7 @@
287	287	.catlinks ul {
288	288	display: inline;
289	289	margin: 0px;
	290	+ padding: 0px;
290	291	list-style: none;
291	292	list-style-type: none;
292	293	list-style-image: none;

Rev.	Commit summary	Author	Date
r97414	1.18: MFT r96775, r97362, r97365, r97381	catrope	11:36, 18 September 2011
r99935	Add 'padding: 0;' for '.catlinks ul' in commonPrint as well...	krinkle	23:51, 15 October 2011

Rev.	Commit summary	Author	Date
r92054	Render category links as an HTML list. Bug 12261. Based on patch by Thana & B...	diebuche	10:41, 13 July 2011

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

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

11:37, 18 September 2011 Catrope (talk | contribs) changed the tags for r97381 [removed: 1.18]
11:10, 18 September 2011 Catrope (talk | contribs) changed the status of r97381 [removed: new added: ok]
18:05, 17 September 2011 Krinkle (talk | contribs) changed the tags for r97381 [added: 1.18]