tighten up linting for Issue #218

bgrins · May 30, 2014 · 45934a3 · 45934a3 · Leyart · Jun 10, 2014
1 parent 117d17f
commit 45934a3
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/Gruntfile.js b/Gruntfile.js
@@ -10,9 +10,9 @@ module.exports = function(grunt) {
 
     jshint: {
       options: {
-        browser: true,
         sub: true,
-
+        strict: true,
+        newcap: false,
         globals: {
           jQuery: true
         }

diff --git a/spectrum.js b/spectrum.js
@@ -4,6 +4,8 @@
 // License: MIT
 
 (function (window, $, undefined) {
+    "use strict";
+
     var defaultOpts = {
 
         // Callbacks
@@ -404,11 +406,11 @@
 
             function palletElementClick(e) {
                 if (e.data && e.data.ignore) {
-                    set($(this).data("color"));
+                    set($(e.target).data("color"));
                     move();
                 }
                 else {
-                    set($(this).data("color"));
+                    set($(e.target).data("color"));
                     move();
                     updateOriginalInput(true);
                     hide();
@@ -1047,8 +1049,6 @@
         };
     }
 
-    function log(){/* jshint -W021 */if(window.console){if(Function.prototype.bind)log=Function.prototype.bind.call(console.log,console);else log=function(){Function.prototype.apply.call(console.log,console,arguments);};log.apply(this,arguments);}}
-
     /**
     * Define a jQuery plugin
     */