Dev: make sure all files can pass JSHint!

diff --git a/grunt.js b/grunt.js
index 91293e6bc7258dc8c160c9513c506a84ffbd5424..524421be79c16fb8b26217605bc04a4912d1f06e 100644 (file)
--- a/grunt.js
+++ b/grunt.js
@@ -285,10 +285,7 @@ grunt.initConfig({
                })
        },
        lint: {
-               ui: grunt.file.expandFiles( "ui/*.js" ).filter(function( file ) {
-                       // TODO remove items from this list once rewritten
-                       return !( /(mouse)\.js$/ ).test( file );
-               }),
+               ui: "ui/*.js",
                grunt: [ "grunt.js", "build/**/*.js" ],
                tests: "tests/unit/**/*.js"
        },

diff --git a/ui/jquery.ui.mouse.js b/ui/jquery.ui.mouse.js
index cdc063f91dd896201bb5bb0458cb5304e68fcf8f..6abd5ceaf66b413bae4ea94076682ab82ec0e53e 100644 (file)
--- a/ui/jquery.ui.mouse.js
+++ b/ui/jquery.ui.mouse.js
@@ -14,7 +14,7 @@
 (function( $, undefined ) {
 
 var mouseHandled = false;
-$( document ).mouseup( function( e ) {
+$( document ).mouseup( function() {
        mouseHandled = false;
 });
 
@@ -111,7 +111,7 @@ $.widget("ui.mouse", {
 
        _mouseMove: function(event) {
                // IE mouseup check - mouseup happened when mouse was out of window
-               if ($.ui.ie && !(document.documentMode >= 9) && !event.button) {
+               if ($.ui.ie && ( !document.documentMode || document.documentMode < 9 ) && !event.button) {
                        return this._mouseUp(event);
                }
 
@@ -155,15 +155,15 @@ $.widget("ui.mouse", {
                );
        },
 
-       _mouseDelayMet: function(event) {
+       _mouseDelayMet: function(/* event */) {
                return this.mouseDelayMet;
        },
 
        // These are placeholder methods, to be overriden by extending plugin
-       _mouseStart: function(event) {},
-       _mouseDrag: function(event) {},
-       _mouseStop: function(event) {},
-       _mouseCapture: function(event) { return true; }
+       _mouseStart: function(/* event */) {},
+       _mouseDrag: function(/* event */) {},
+       _mouseStop: function(/* event */) {},
+       _mouseCapture: function(/* event */) { return true; }
 });
 
 })(jQuery);