style: fix the grunt watch and make jshint happy

Author: douglasduteil

demo/demo.js

@@ -1,3 +1,5 @@
+'use strict';
+
 angular.module('doc.ui-ace', ['ui.ace', 'prettifyDirective', 'ui.bootstrap', 'plunker'])
   .controller('AceCtrl', ['$scope', function ($scope) {
 
@@ -13,7 +15,7 @@ angular.module('doc.ui-ace', ['ui.ace', 'prettifyDirective', 'ui.bootstrap', 'pl
 
         // HACK to have the ace instance in the scope...
         $scope.modeChanged = function () {
-          _ace.getSession().setMode("ace/mode/" + $scope.mode.toLowerCase());
+          _ace.getSession().setMode('ace/mode/' + $scope.mode.toLowerCase());
         };
 
       }
@@ -28,4 +30,4 @@ angular.module('doc.ui-ace', ['ui.ace', 'prettifyDirective', 'ui.bootstrap', 'pl
       'function foo(msg) {\n\tvar r = Math.random();\n\treturn "" + r + " : " + msg;\n}';
 
   }])
-;
+;

gruntFile.js

@@ -1,3 +1,5 @@
+'use strict';
+
 module.exports = function (grunt) {
 
   var _ = grunt.util._;
@@ -6,7 +8,7 @@ module.exports = function (grunt) {
 
   // Default task.
   grunt.registerTask('default', ['jshint', 'karma:unit']);
-  grunt.registerTask('server', ['karma:start']);
+  grunt.registerTask('serve', ['karma:continuous', 'watch']);
   grunt.registerTask('build-doc', ['uglify', 'copy']);
 
   var testConfig = function (configFile, customOptions) {
@@ -35,26 +37,27 @@ module.exports = function (grunt) {
         ' */',
         ''].join('\n'),
       view : {
-        humaName : "UI Ace",
-        repoName : "ui-ace",
-        demoHTML : grunt.file.read("demo/demo.html"),
-        demoJS : grunt.file.read("demo/demo.js"),
+        humaName : 'UI Ace',
+        repoName : 'ui-ace',
+        demoHTML : grunt.file.read('demo/demo.html'),
+        demoJS : grunt.file.read('demo/demo.js'),
         css: ['assets/css/demo.css'],
         js: js_dependencies.concat(['build/ui-ace.min.js'])
       }
     },
     watch: {
       karma: {
-        files: ['ui-ace.js', 'test/*.js'],
-        tasks: ['karma:unit:run'] //NOTE the :run flag
+        files: ['ui-ace.js', 'demo/**/*.js', 'gruntFile.js'],
+        tasks: ['jshint', 'karma:unit:run'] //NOTE the :run flag
       }
     },
     karma: {
       unit: testConfig('test/karma.conf.js'),
-      start: {configFile: 'test/karma.conf.js'}
+      start: {configFile: 'test/karma.conf.js'},
+      continuous: {configFile: 'test/karma.conf.js', background: true }
     },
     jshint:{
-      files:['ui-ace.js', 'demo/**/*.js'],
+      files:['ui-ace.js', 'demo/**/*.js', 'gruntFile.js'],
       options: { jshintrc: '.jshintrc' }
     },
     uglify: {

ui-ace.js

@@ -1,10 +1,8 @@
+'use strict';
+
 /**
  * Binds a ACE Ediitor widget
  */
-
-//TODO handle Could not load worker ace.js:1
-//DOMException {message: "SECURITY_ERR: DOM Exception 18", name: "SECURITY_ERR", code: 18, stack: "Error: An attempt was made to break through the se…cloudfront.net/src-min-noconflict/ace.js:1:76296)", INDEX_SIZE_ERR: 1…}
-
 angular.module('ui.ace', [])
   .constant('uiAceConfig', {})
   .directive('uiAce', ['uiAceConfig', function (uiAceConfig) {
@@ -66,16 +64,16 @@ angular.module('ui.ace', [])
 
         // Basic options
         if (angular.isString(opts.theme)) {
-          acee.setTheme("ace/theme/" + opts.theme);
+          acee.setTheme('ace/theme/' + opts.theme);
         }
         if (angular.isString(opts.mode)) {
-          session.setMode("ace/mode/" + opts.mode);
+          session.setMode('ace/mode/' + opts.mode);
         }
 
         attrs.$observe('readonly', function (value) {
           acee.setReadOnly(value === 'true');
         });
-        
+
         // Value Blind
         if (angular.isDefined(ngModel)) {
           ngModel.$formatters.push(function (value) {
@@ -98,4 +96,4 @@ angular.module('ui.ace', [])
 
       }
     };
-  }]);
+  }]);