"Catrope" changed the status of MediaWiki.r106402 to "fixme" and commented it.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/106402#c28341

Old Status: new
> New Status: fixme

Commit summary for MediaWiki.r106402:

removeing wikipedia specific language from tooltip, remove document.ready wrap 
on getUserInfo method & used in callback in inject method, change order of 
which moodbar is initialized so that mediawiki.util is available in init 
script. follow up to r105613

Catrope's comment:

<pre>
+       'moodbar-tooltip-title' => 'Let us know about your experience editing 
{{SITENAME}}.',
</pre>
Are you sure this works correctly? moodbar-tooltip-title is used in JS, and 
<nowiki>{{SITENAME}}</nowiki> doesn't work in JS messages. From reading the 
code I'm 90% sure this is broken.

_______________________________________________
MediaWiki-CodeReview mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview

Reply via email to