Siebrand has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/102645


Change subject: Define showMessageGroupStats before it is used
......................................................................

Define showMessageGroupStats before it is used

Change-Id: I54776efb82f05b2eb1043b31b21d062a8b9c63bc
---
M resources/js/ext.translate.mainpage.js
1 file changed, 39 insertions(+), 39 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/TwnMainPage 
refs/changes/45/102645/1

diff --git a/resources/js/ext.translate.mainpage.js 
b/resources/js/ext.translate.mainpage.js
index 0f80409..46f03c8 100644
--- a/resources/js/ext.translate.mainpage.js
+++ b/resources/js/ext.translate.mainpage.js
@@ -10,45 +10,6 @@
        var getUrl = mw.util.getUrl || mw.util.wikiGetlink;
 
        /**
-        * If the source language of all projects is the same as the target 
language,
-        * provide a language selector to select the target language above the 
tiles.
-        */
-       function setupTargetLanguageSelector() {
-               var $sameLanguageULSTrigger,
-                       sourceLanguage = $( '.twn-mainpage-project-tiles' 
).data( 'sourcelanguage' ),
-                       sameLanguageULSLanguages = $.extend( {}, mw.config.get( 
'wgULSLanguages' ) );
-
-               $sameLanguageULSTrigger = $( '<div>' )
-                       .addClass( 'row' )
-                       .append( $( '<div>' )
-                               .addClass( 'two columns' )
-                               .append(
-                                       $( '<button>' )
-                                               .text( mw.msg( 
'twnmp-select-target-language' ) )
-                                               .addClass( 
'same-language-uls-trigger' )
-                               )
-                       );
-
-               $( '.twn-mainpage-project-tiles' ).before( 
$sameLanguageULSTrigger );
-
-               delete sameLanguageULSLanguages[sourceLanguage];
-               $sameLanguageULSTrigger.uls( {
-                       languages: sameLanguageULSLanguages,
-                       top: '20%',
-                       onSelect: showMessageGroupStats,
-                       quickList: function () {
-                               var frequentLanguageList = 
mw.uls.getFrequentLanguageList(),
-                                       sourceLanguagePosition = $.inArray( 
sourceLanguage, frequentLanguageList );
-
-                               // Remove the source language from the usual 
common languages list
-                               frequentLanguageList.splice( 
sourceLanguagePosition, 1 );
-
-                               return frequentLanguageList;
-                       }
-               } );
-       }
-
-       /**
         * Show the message group stats bars.
         */
        function showMessageGroupStats( language ) {
@@ -110,6 +71,45 @@
        }
 
        /**
+        * If the source language of all projects is the same as the target 
language,
+        * provide a language selector to select the target language above the 
tiles.
+        */
+       function setupTargetLanguageSelector() {
+               var $sameLanguageULSTrigger,
+                       sourceLanguage = $( '.twn-mainpage-project-tiles' 
).data( 'sourcelanguage' ),
+                       sameLanguageULSLanguages = $.extend( {}, mw.config.get( 
'wgULSLanguages' ) );
+
+               $sameLanguageULSTrigger = $( '<div>' )
+                       .addClass( 'row' )
+                       .append( $( '<div>' )
+                               .addClass( 'two columns' )
+                               .append(
+                                       $( '<button>' )
+                                               .text( mw.msg( 
'twnmp-select-target-language' ) )
+                                               .addClass( 
'same-language-uls-trigger' )
+                               )
+                       );
+
+               $( '.twn-mainpage-project-tiles' ).before( 
$sameLanguageULSTrigger );
+
+               delete sameLanguageULSLanguages[sourceLanguage];
+               $sameLanguageULSTrigger.uls( {
+                       languages: sameLanguageULSLanguages,
+                       top: '20%',
+                       onSelect: showMessageGroupStats,
+                       quickList: function () {
+                               var frequentLanguageList = 
mw.uls.getFrequentLanguageList(),
+                                       sourceLanguagePosition = $.inArray( 
sourceLanguage, frequentLanguageList );
+
+                               // Remove the source language from the usual 
common languages list
+                               frequentLanguageList.splice( 
sourceLanguagePosition, 1 );
+
+                               return frequentLanguageList;
+                       }
+               } );
+       }
+
+       /**
         * Setup the project tiles in the main page.
         */
        function setupProjectTiles() {

-- 
To view, visit https://gerrit.wikimedia.org/r/102645
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I54776efb82f05b2eb1043b31b21d062a8b9c63bc
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/TwnMainPage
Gerrit-Branch: master
Gerrit-Owner: Siebrand <[email protected]>

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

Reply via email to