r79605 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r79604‎ | r79605 | r79606 >
Date:21:57, 4 January 2011
Author:krinkle
Status:deferred
Tags:
Comment:
Adding missing semicolons. (JSLint fixes for Mozilla)
Modified paths:
  • /trunk/extensions/Translate/js/translate.langstats.js (modified) (history)

Diff [purge]

Index: trunk/extensions/Translate/js/translate.langstats.js
@@ -7,7 +7,7 @@
88 jQuery( document ).ready( function() {
99
1010 var $translateTable = $( '.mw-sp-translate-table' ),
11 - $metaRows = $( 'tr[data-ismeta=1]', $translateTable ),
 11+ $metaRows = $( 'tr[data-ismeta=1]', $translateTable );
1212
1313 // Only do stuff if there are any meta group rows on this pages
1414 if ( $metaRows.size() ) {
@@ -46,17 +46,17 @@
4747 $allToggles_cache = null,
4848 $toggleAllButton = $( '<span class="mw-sp-langstats-expander">[</span>' ).append( $( '<a href="#" onclick="return false;">' ).text( mw.msg( 'translate-langstats-expandall' ) ) ).append( ']' ).click( function() {
4949 var $el = $( this ),
50 - $allToggles = !!$allToggles_cache ? $allToggles_cache : $( '.mw-sp-langstats-toggle', $translateTable ),
 50+ $allToggles = !!$allToggles_cache ? $allToggles_cache : $( '.mw-sp-langstats-toggle', $translateTable );
5151 // Switch the state and toggle the rows
5252 // and update the local toggles too
5353 if ( $el.hasClass( 'mw-sp-langstats-expander' ) ) {
5454 $allChildRows.show();
55 - $el.add( $allToggles ).removeClass( 'mw-sp-langstats-expander' ).addClass( 'mw-sp-langstats-collapser' )
 55+ $el.add( $allToggles ).removeClass( 'mw-sp-langstats-expander' ).addClass( 'mw-sp-langstats-collapser' );
5656 $el.find( '> a' ).text( mw.msg( 'translate-langstats-collapseall' ) );
5757 $allToggles.find( '> a' ).text( mw.msg( 'translate-langstats-collapse' ) );
5858 } else {
5959 $allChildRows.hide();
60 - $el.add( $allToggles ).addClass( 'mw-sp-langstats-expander' ).removeClass( 'mw-sp-langstats-collapser' )
 60+ $el.add( $allToggles ).addClass( 'mw-sp-langstats-expander' ).removeClass( 'mw-sp-langstats-collapser' );
6161 $el.find( '> a' ).text( mw.msg( 'translate-langstats-expandall' ) );
6262 $allToggles.find( '> a' ).text( mw.msg( 'translate-langstats-expand' ) );
6363 }

Status & tagging log