r74277 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:< r74276‎ | r74277 | r74278 >
Date:21:12, 4 October 2010
Author:kaldari
Status:deferred (Comments)
Tags:
Comment:
switching to icon on bits for better security/stability, per comment at r74110
Modified paths:
  • /trunk/extensions/CentralNotice/centralnotice.js (modified) (history)

Diff [purge]

Index: trunk/extensions/CentralNotice/centralnotice.js
——@@ -26,7 +26,7 @@
2727 var buttonValue = '[<a href="#" onclick="toggleNotice();return false">{{int:centralnotice-shared-hide}}</a>]';
2828 break;
2929 case 'close':
30 - var buttonValue = '<a href="#" onclick="toggleNotice();return false"><img border="0" src="http://upload.wikimedia.org/wikipedia/commons/1/1b/Close_window_X.gif" width="10" height="10" alt="Close" /></a>';
 30+ var buttonValue = '<a href="#" onclick="toggleNotice();return false"><img border="0" src="http://bits.wikimedia.org/skins-1.5/common/closewindow.png" width="11" height="11" alt="Close" /></a>';
3131 break;
3232 }
3333 if (document.selection) {

Follow-up revisions

RevisionCommit summaryAuthorDate
r74320fix for r74277 and r74110kaldari18:57, 5 October 2010
r79696removing wikimedia specific stuff per r74110, r74277, and r74320kaldari00:56, 6 January 2011

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r74110add in close button stuffkaldari21:27, 1 October 2010

Comments

#Comment by Catrope (talk | contribs)   13:09, 5 October 2010

The URL should not be hardcoded like that. Use the stylepath JS variable (on WMF this is set to 'http:/bits.wikimedia.org/skins-1.5'

#Comment by Kaldari (talk | contribs)   18:58, 5 October 2010

fixed in r74320.

#Comment by Catrope (talk | contribs)   13:11, 5 October 2010

Depends on core change (added icon) in r74275.

Status & tagging log