AngularJS + require.js: jshint style

6e43c8dd · Pascal Hartig · 9b5e8b15 · 6e43c8dd · 6e43c8dd · 6e43c8dd
Commit 6e43c8dd authored Feb 24, 2013 by Pascal Hartig
7 changed files
--- a/labs/dependency-examples/angularjs_require/index.html
+++ b/labs/dependency-examples/angularjs_require/index.html
 <!doctype html>
 <html lang="en">
-<head>
+	<head>
 		<meta charset="utf-8">
 		<meta http-equiv="X-UA-Compatible" content="IE=edge,chrome=1">
 		<title>AngularJS • TodoMVC</title>
@@ -16,9 +16,8 @@
 		   (c) 2010-2012 Google, Inc. http://angularjs.org
 		   License: MIT
 		 */
-    (function(i){'use strict';function d(c,b,e){return c[b]||(c[b]=e())}return d(d(i,"angular",Object),"module",function(){var c={};return function(b,e,f){e&&c.hasOwnProperty(b)&&(c[b]=null);return d(c,b,function(){function a(a,b,d){return function(){c[d||"push"]([a,b,arguments]);return g}}if(!e)throw Error("No module: "+b);var c=[],d=[],h=a("$injector","invoke"),g={_invokeQueue:c,_runBlocks:d,requires:e,name:b,provider:a("$provide","provider"),factory:a("$provide","factory"),service:a("$provide","service"),
-    value:a("$provide","value"),constant:a("$provide","constant","unshift"),filter:a("$filterProvider","register"),controller:a("$controllerProvider","register"),directive:a("$compileProvider","directive"),config:h,run:function(a){d.push(a);return this}};f&&h(f);return g})}})})(window);
-  </script>
+		(function(i){'use strict';function d(c,b,e){return c[b]||(c[b]=e())}return d(d(i,"angular",Object),"module",function(){var c={};return function(b,e,f){e&&c.hasOwnProperty(b)&&(c[b]=null);return d(c,b,function(){function a(a,b,d){return function(){c[d||"push"]([a,b,arguments]);return g}}if(!e)throw Error("No module: "+b);var c=[],d=[],h=a("$injector","invoke"),g={_invokeQueue:c,_runBlocks:d,requires:e,name:b,provider:a("$provide","provider"),factory:a("$provide","factory"),service:a("$provide","service"), value:a("$provide","value"),constant:a("$provide","constant","unshift"),filter:a("$filterProvider","register"),controller:a("$controllerProvider","register"),directive:a("$compileProvider","directive"),config:h,run:function(a){d.push(a);return this}};f&&h(f);return g})}})})(window);
+</script>
 </head>
 <body>
 	<section id="todoapp" ng-controller="TodoController">

--- a/labs/dependency-examples/angularjs_require/js/app.js
+++ b/labs/dependency-examples/angularjs_require/js/app.js
+/*global define*/
 'use strict';

 define(['angular'], function (angular) {

--- a/labs/dependency-examples/angularjs_require/js/controllers/todo.js
+++ b/labs/dependency-examples/angularjs_require/js/controllers/todo.js
+/*global define*/
 'use strict';

 /**
@@ -8,31 +9,34 @@

 define(['app', 'services/todoStorage'], function (app) {
 	return app.controller('TodoController', ['$scope', '$location', 'todoStorage', 'filterFilter',
-    function TodoController( $scope, $location, todoStorage, filterFilter ) {
+		function TodoController($scope, $location, todoStorage, filterFilter) {
 			var todos = $scope.todos = todoStorage.get();

-      $scope.newTodo = "";
+			$scope.newTodo = '';
 			$scope.editedTodo = null;

-      $scope.$watch('todos', function() {
+			$scope.$watch('todos', function () {
 				$scope.remainingCount = filterFilter(todos, {completed: false}).length;
 				$scope.doneCount = todos.length - $scope.remainingCount;
-        $scope.allChecked = !$scope.remainingCount
+				$scope.allChecked = !$scope.remainingCount;
 				todoStorage.put(todos);
 			}, true);

-      if ( $location.path() === '' ) $location.path('/');
+			if ($location.path() === '') {
+				$location.path('/');
+			}
+
 			$scope.location = $location;

-      $scope.$watch( 'location.path()', function( path ) {
-        $scope.statusFilter = (path == '/active') ?
-          { completed: false } : (path == '/completed') ?
+			$scope.$watch('location.path()', function (path) {
+				$scope.statusFilter = (path === '/active') ?
+					{ completed: false } : (path === '/completed') ?
 					{ completed: true } : null;
 			});


-      $scope.addTodo = function() {
-        if ( !$scope.newTodo.length ) {
+			$scope.addTodo = function () {
+				if (!$scope.newTodo.length) {
 					return;
 				}

@@ -45,33 +49,33 @@ define(['app', 'services/todoStorage'], function (app) {
 			};


-      $scope.editTodo = function( todo ) {
+			$scope.editTodo = function (todo) {
 				$scope.editedTodo = todo;
 			};


-      $scope.doneEditing = function( todo ) {
+			$scope.doneEditing = function (todo) {
 				$scope.editedTodo = null;
-        if ( !todo.title ) {
+				if (!todo.title) {
 					$scope.removeTodo(todo);
 				}
 			};


-      $scope.removeTodo = function( todo ) {
+			$scope.removeTodo = function (todo) {
 				todos.splice(todos.indexOf(todo), 1);
 			};


-      $scope.clearDoneTodos = function() {
-        $scope.todos = todos = todos.filter(function( val ) {
+			$scope.clearDoneTodos = function () {
+				$scope.todos = todos = todos.filter(function (val) {
 					return !val.completed;
 				});
 			};


-      $scope.markAll = function( done ) {
-        todos.forEach(function( todo ) {
+			$scope.markAll = function (done) {
+				todos.forEach(function (todo) {
 					todo.completed = done;
 				});
 			};

--- a/labs/dependency-examples/angularjs_require/js/directives/todoBlur.js
+++ b/labs/dependency-examples/angularjs_require/js/directives/todoBlur.js
+/*global define*/
 'use strict';
 /**
 * Directive that executes an expression when the element it is applied to loses focus.
 */

-define(['app'], function ( app ) {
-  app.directive('todoBlur', function() {
-    return function( scope, elem, attrs ) {
-      elem.bind('blur', function() {
+define(['app'], function (app) {
+	app.directive('todoBlur', function () {
+		return function (scope, elem, attrs) {
+			elem.bind('blur', function () {
 				scope.$apply(attrs.todoBlur);
 			});
 		};

--- a/labs/dependency-examples/angularjs_require/js/directives/todoFocus.js
+++ b/labs/dependency-examples/angularjs_require/js/directives/todoFocus.js
+/*global define*/
 'use strict';
 /**
 * Directive that places focus on the element it is applied to when the expression it binds to evaluates to true.
 */
-define(['app'], function ( app ) {
-  app.directive('todoFocus', ['$timeout', function( $timeout ) {
-    return function( scope, elem, attrs ) {
-      scope.$watch(attrs.todoFocus, function( newval ) {
-        if ( newval ) {
-          $timeout(function() {
+define(['app'], function (app) {
+	app.directive('todoFocus', ['$timeout', function ($timeout) {
+		return function (scope, elem, attrs) {
+			scope.$watch(attrs.todoFocus, function (newval) {
+				if (newval) {
+					$timeout(function () {
 						elem[0].focus();
 					}, 0, false);
 				}

--- a/labs/dependency-examples/angularjs_require/js/main.js
+++ b/labs/dependency-examples/angularjs_require/js/main.js
+/*global require*/
 'use strict';

 require.config({

--- a/labs/dependency-examples/angularjs_require/js/services/todoStorage.js
+++ b/labs/dependency-examples/angularjs_require/js/services/todoStorage.js
+/*global define*/
 'use strict';

 /**
 * Services that persists and retrieves TODOs from localStorage.
 */
 define(['app'], function (app) {
-  app.factory( 'todoStorage', function() {
+	app.factory('todoStorage', function () {
 		var STORAGE_ID = 'todos-angularjs-requirejs';

 		return {
-      get: function() {
+			get: function () {
 				return JSON.parse(localStorage.getItem(STORAGE_ID) || '[]');
 			},

-      put: function( todos ) {
+			put: function (todos) {
 				localStorage.setItem(STORAGE_ID, JSON.stringify(todos));
 			}
 		};