r78424 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r78423‎ | r78424 | r78425 >
Date:01:03, 15 December 2010
Author:happy-melon
Status:ok
Tags:
Comment:
Follow-up r77796: a couple of typos in the comment, and break more consistently at 90 cols.
Modified paths:
  • /trunk/phase3/includes/DefaultSettings.php (modified) (history)

Diff [purge]

Index: trunk/phase3/includes/DefaultSettings.php
@@ -2288,15 +2288,14 @@
22892289 * of new blocks in monobook. vector and modern should work without any special css.
22902290 *
22912291 * $wgFooterIcons itself is a key/value array.
2292 - * The key os the name of a block that the icons will be wrapped in.
2293 - * The final id varries by skin; Monobook and Vector will turn poweredby into f-poweredbyico
2294 - * while Modern turns it into mw_poweredby.
2295 - * The value is a key/value array of icons. The key may or may not be used by the
2296 - * skin but it can be used to find the icon and unset it or change the icon if needed.
2297 - * This is useful for disabling icons that are set by extensions.
2298 - * The value should be either a string or an array.
2299 - * If it is a string it will be output directly, however some skins may choose to ignore it.
2300 - * An array is the preferred format for the icon, the following keys are used:
 2292+ * The key is the name of a block that the icons will be wrapped in. The final id varies
 2293+ * by skin; Monobook and Vector will turn poweredby into f-poweredbyico while Modern
 2294+ * turns it into mw_poweredby. The value is a key/value array of icons. The key may or
 2295+ * may not be used by the skin but it can be used to find the icon and unset it or
 2296+ * change the icon if needed. This is useful for disabling icons that are set by extensions.
 2297+ * The value should be either a string or an array. If it is a string it will be output
 2298+ * directly, however some skins may choose to ignore it. An array is the preferred format
 2299+ * for the icon, the following keys are used:
23012300 * src: An absolute url to the image to use for the icon, this is recommended
23022301 * but not required, however some skins will ignore icons without an image
23032302 * url: The url to use in the <a> arround the text or icon, if not set an <a> will not be outputted

Follow-up revisions

RevisionCommit summaryAuthorDate
r784371.17: Merge tagged revisions from trunk: r77878, r77981, r77982, r77994, r780...catrope14:14, 15 December 2010

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r77796Followup r77793, documentation for icon as a string was missing.dantman11:16, 5 December 2010

Status & tagging log