build: Update eslint to 0.4.0 and make pass

[lhc/web/wiklou.git] / resources / src / mediawiki / mediawiki.checkboxtoggle.js

diff --git a/resources/src/mediawiki/mediawiki.checkboxtoggle.js b/resources/src/mediawiki/mediawiki.checkboxtoggle.js
index 62760d2..901f875 100644 (file)
--- a/resources/src/mediawiki/mediawiki.checkboxtoggle.js
+++ b/resources/src/mediawiki/mediawiki.checkboxtoggle.js
@@ -13,28 +13,29 @@
        'use strict';
 
        $( function () {
-               var $checkboxes = $( 'li input[type=checkbox]' );
+               // FIXME: This shouldn't be a global selector to avoid conflicts
+               // with unrelated content on the same page. (T131318)
+               var $checkboxes = $( 'li input[type="checkbox"]' );
 
                function selectAll( check ) {
                        $checkboxes.prop( 'checked', check );
                }
 
                $( '.mw-checkbox-all' ).click( function ( e ) {
-                       selectAll( true );
                        e.preventDefault();
+                       selectAll( true );
                } );
                $( '.mw-checkbox-none' ).click( function ( e ) {
-                       selectAll( false );
                        e.preventDefault();
+                       selectAll( false );
                } );
                $( '.mw-checkbox-invert' ).click( function ( e ) {
-                       $checkboxes.each( function () {
-                               $( this ).prop( 'checked', !$( this ).is( ':checked' ) );
-                       } );
                        e.preventDefault();
+                       $checkboxes.prop( 'checked', function ( i, val ) {
+                               return !val;
+                       } );
                } );
 
        } );
 
 }( mediaWiki, jQuery ) );
-