Small tweaks to jQuery example

sindresorhus · sindresorhus · commit 4d837fb91a79 · 2012-03-01T14:12:13.000+01:00
- Use the base.css
- Trim whitespace on new todo
- Trim whitespace on todo update, and remove todo if empty
- Reset the checked state of "Mark all as complete", when the "Clear completed" button is clicked
diff --git a/architecture-examples/jquery/css/app.css b/architecture-examples/jquery/css/app.css
diff --git a/architecture-examples/jquery/index.html b/architecture-examples/jquery/index.html
@@ -3,7 +3,7 @@
 <head>
 	<meta charset="utf-8">
 	<title>jQuery - TodoMVC</title>
-	<link rel="stylesheet" href="css/app.css">
+	<link rel="stylesheet" href="../../assets/base.css">
 </head>
 <body>
 	<div id="todoapp">
diff --git a/architecture-examples/jquery/js/app.js b/architecture-examples/jquery/js/app.js
@@ -97,6 +97,7 @@ jQuery(function($) {
 					todos.splice( l, 1 );
 				}
 			}
+			App.$toggleAll.attr( 'checked', false );
 			App.render();
 		},
 		// Accepts an element from inside the ".item" div and returns the corresponding todo in the todos array.
@@ -110,16 +111,13 @@ jQuery(function($) {
 			});
 		},
 		create: function(e) {
-			if ( e.which !== App.ENTER_KEY ) {
-				return;
-			}
 			var $input = $(this),
-				inputVal = $input.val();
-			if ( !inputVal ) {
+				val = $.trim( $input.val() );
+			if ( e.which !== App.ENTER_KEY || !val ) {
 				return;
 			}
 			App.todos.push({
-				title: inputVal,
+				title: val,
 				id: Utils.uuid(),
 				done: false
 			});
@@ -143,9 +141,13 @@ jQuery(function($) {
 		update: function() {
 			var val = $(this).removeClass('editing').val();
 			App.getTodo( this, function(i) {
-				this.todos[i].title = val;
+				if ( val ) {
+					this.todos[i].title = val;
+				} else {
+					this.todos.splice( i, 1 );
+				}
+				this.render();
 			});
-			App.render();
 		},
 		destroy: function() {
 			App.getTodo( this, function(i) {
