CanJS: jshint style

ccad8ec9 · Pascal Hartig · 8c233266 · ccad8ec9 · ccad8ec9 · ccad8ec9
Commit ccad8ec9 authored Feb 20, 2013 by Pascal Hartig
4 changed files
--- a/architecture-examples/canjs/index.html
+++ b/architecture-examples/canjs/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>CanJS • TodoMVC</title>
@@ -8,24 +8,24 @@
 		<!--[if IE]>
 		<script src="../../assets/ie.js"></script>
 		<![endif]-->
-</head>
-<body>
-<section id="todoapp">
-</section>
-<div id="info">
+	</head>
+	<body>
+		<section id="todoapp">
+		</section>
+		<div id="info">
 			<p>Double-click to edit a todo</p>
 			<p>Written by <a href="http://bitovi.com">Bitovi</a></p>
 			<p>Part of <a href="http://todomvc.com">TodoMVC</a></p>
-</div>
+		</div>

-<script src="../../assets/jquery.min.js"></script>
-<script src="../../assets/base.js"></script>
+		<script src="../../assets/jquery.min.js"></script>
+		<script src="../../assets/base.js"></script>

-<script src="js/lib/can.jquery-1.1.0.min.js"></script>
-<script src="js/lib/can.mustache.min.js"></script>
-<script src="js/lib/can.localstorage.min.js"></script>
-<script src="js/models/todo.js"></script>
-<script src="js/todos/todos.js"></script>
-<script src="js/app.js"></script>
-</body>
+		<script src="js/lib/can.jquery-1.1.0.min.js"></script>
+		<script src="js/lib/can.mustache.min.js"></script>
+		<script src="js/lib/can.localstorage.min.js"></script>
+		<script src="js/models/todo.js"></script>
+		<script src="js/todos/todos.js"></script>
+		<script src="js/app.js"></script>
+	</body>
 </html>
--- a/architecture-examples/canjs/js/app.js
+++ b/architecture-examples/canjs/js/app.js
-(function() {
-	$(function() {
+/*global $ Todos Models Mustache can*/
+(function () {
+	'use strict';
+	$(function () {
 		// Set up a route that maps to the `filter` attribute
-		can.route( ':filter' );
+		can.route(':filter');
 		// Delay routing until we initialized everything
 		can.route.ready(false);

 		// View helper for pluralizing strings
-		Mustache.registerHelper('plural', function(str, count) {
+		Mustache.registerHelper('plural', function (str, count) {
 			return str + (count !== 1 ? 's' : '');
 		});

 		// Initialize the app
-		Models.Todo.findAll({}, function(todos) {
+		Models.Todo.findAll({}, function (todos) {
 			new Todos('#todoapp', {
 				todos: todos,
-				state : can.route,
+				state: can.route,
 				view : 'views/todos.mustache'
 			});
 		});

--- a/architecture-examples/canjs/js/models/todo.js
+++ b/architecture-examples/canjs/js/models/todo.js
@@ -9,30 +9,30 @@
 	}, {
 		// Returns if this instance matches a given filter
 		// (currently `active` and `complete`)
-		matches : function() {
+		matches : function () {
 			var filter = can.route.attr('filter');
-			return !filter || (filter === 'active' && !this.attr('complete'))
-				|| (filter === 'completed' && this.attr('complete'));
+			return !filter || (filter === 'active' && !this.attr('complete')) ||
+				(filter === 'completed' && this.attr('complete'));
 		}
 	});

 	// List for Todos
 	Todo.List = can.Model.List({
-		completed: function() {
+		completed: function () {
 			var completed = 0;

-			this.each(function(todo) {
+			this.each(function (todo) {
 				completed += todo.attr('complete') ? 1 : 0;
 			});

 			return completed;
 		},

-		remaining: function() {
+		remaining: function () {
 			return this.attr('length') - this.completed();
 		},

-		allComplete: function() {
+		allComplete: function () {
 			return this.attr('length') === this.completed();
 		}
 	});

--- a/architecture-examples/canjs/js/todos/todos.js
+++ b/architecture-examples/canjs/js/todos/todos.js
-/*global can, Models, $ */
+/*global can Models*/
 (function (namespace, undefined) {
 	'use strict';

@@ -10,13 +10,13 @@
 		}
 	}, {
 		// Initialize the Todos list
-		init: function() {
+		init: function () {
 			// Render the Todos
 			this.element.append(can.view(this.options.view, this.options));
 		},

 		// Listen for when a new Todo has been entered
-		'#new-todo keyup': function(el, e) {
+		'#new-todo keyup': function (el, e) {
 			var value = can.trim(el.val());
 			if (e.keyCode === ENTER_KEY && value !== '') {
 				new Models.Todo({
@@ -29,33 +29,35 @@
 		},

 		// Handle a newly created Todo
-		'{Models.Todo} created': function(list, e, item) {
+		'{Models.Todo} created': function (list, e, item) {
 			this.options.todos.push(item);
 			// Reset the filter so that you always see your new todo
 			this.options.state.attr('filter', '');
 		},

 		// Listener for when the route changes
-		'{state} change' : function() {
+		'{state} change' : function () {
 			// Remove the `selected` class from the old link and add it to the link for the current location hash
 			this.element.find('#filters').find('a').removeClass('selected')
 				.end().find('[href="' + window.location.hash + '"]').addClass('selected');
 		},

 		// Listen for editing a Todo
-		'.todo dblclick': function(el) {
-			el.data('todo').attr('editing', true).save(function() {
+		'.todo dblclick': function (el) {
+			el.data('todo').attr('editing', true).save(function () {
 				el.children('.edit').focus();
 			});
 		},

 		// Update a todo
-		updateTodo: function(el) {
+		updateTodo: function (el) {
 			var value = can.trim(el.val()),
 				todo = el.closest('.todo').data('todo');

 			// If we don't have a todo we don't need to do anything
-			if(!todo) return;
+			if (!todo) {
+				return;
+			}

 			if (value === '') {
 				todo.destroy();
@@ -68,36 +70,36 @@
 		},

 		// Listen for an edited Todo
-		'.todo .edit keyup': function(el, e) {
+		'.todo .edit keyup': function (el, e) {
 			if (e.keyCode === ENTER_KEY) {
 				this.updateTodo(el);
 			}
 		},

-		'.todo .edit focusout' : "updateTodo",
+		'.todo .edit focusout' : 'updateTodo',

 		// Listen for the toggled completion of a Todo
-		'.todo .toggle click': function(el) {
+		'.todo .toggle click': function (el) {
 			el.closest('.todo').data('todo')
 				.attr('complete', el.is(':checked'))
 				.save();
 		},

 		// Listen for a removed Todo
-		'.todo .destroy click': function(el) {
+		'.todo .destroy click': function (el) {
 			el.closest('.todo').data('todo').destroy();
 		},

 		// Listen for toggle all completed Todos
 		'#toggle-all click': function (el) {
 			var toggle = el.prop('checked');
-			can.each(this.options.todos, function(todo) {
+			can.each(this.options.todos, function (todo) {
 				todo.attr('complete', toggle).save();
 			});
 		},

 		// Listen for removing all completed Todos
-		'#clear-completed click': function() {
+		'#clear-completed click': function () {
 			for (var i = this.options.todos.length - 1, todo; i > -1 && (todo = this.options.todos[i]); i--) {
 				if (todo.attr('complete')) {
 					todo.destroy();