r78346 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r78345‎ | r78346 | r78347 >
Date:00:01, 14 December 2010
Author:pdhanda
Status:ok
Tags:
Comment:
Follow up to r77897. Added some style to language links.
Modified paths:
  • /trunk/extensions/FlaggedRevs/FlaggedRevs.php (modified) (history)
  • /trunk/extensions/FlaggedRevs/client/flaggedrevs.css (modified) (history)
  • /trunk/extensions/FlaggedRevs/client/flaggedrevs.js (modified) (history)

Diff [purge]

Index: trunk/extensions/FlaggedRevs/FlaggedRevs.php
@@ -247,7 +247,7 @@
248248 $wgAvailableRights[] = 'stablesettings';
249249
250250 # Bump this number every time you change flaggedrevs.css/flaggedrevs.js
251 -$wgFlaggedRevStyleVersion = 83;
 251+$wgFlaggedRevStyleVersion = 84;
252252
253253 $wgExtensionFunctions[] = 'efLoadFlaggedRevs';
254254
Index: trunk/extensions/FlaggedRevs/client/flaggedrevs.css
@@ -342,3 +342,9 @@
343343 margin-top: -1.3em;
344344 float: left;
345345 }
 346+
 347+div.langlinks {
 348+ margin-top: 1em;
 349+ padding: 5px;
 350+ border: 1px solid #aaa;
 351+}
Index: trunk/extensions/FlaggedRevs/client/flaggedrevs.js
@@ -215,7 +215,7 @@
216216 }
217217 languageshtml = jQuery(result).find("languageshtml");
218218 if ( languageshtml && languageshtml.text() ) {
219 - contentsDiv.innerHTML += languageshtml.text();
 219+ contentsDiv.innerHTML += "<div class='langlinks' >" + languageshtml.text() + "</div>";
220220 }
221221
222222 },

Follow-up revisions

RevisionCommit summaryAuthorDate
r78348Merged -r78346 from trunkpdhanda00:02, 14 December 2010
r80733MFT first round of FlaggedRevs fixes r78517, r78346, r78319, r78293, r78226, ...demon23:56, 21 January 2011

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r77897Part of fix/workaround to bug 26163. Get the categories and languagelinks as...pdhanda19:47, 6 December 2010

Status & tagging log