angularjs-perf: Removed todo-escaped directive

Using the native `ng-keydown` instead

examples/angularjs-perf/index.html

@@ -19,14 +19,20 @@
 				<input id="toggle-all" type="checkbox" ng-model="TC.allChecked" ng-click="TC.markAll(TC.allChecked)">
 				<label for="toggle-all">Mark all as complete</label>
 				<ul id="todo-list">
-					<li ng-repeat="todo in TC.todos | filter:TC.statusFilter track by $index" ng-class="{completed: todo.completed, editing: todo === TC.editedTodo}">
+					<li ng-repeat="todo in TC.todos | filter:TC.statusFilter track by $index" 
+						ng-class="{completed: todo.completed, editing: todo === TC.editedTodo}">
 						<div class="view">
 							<input class="toggle" type="checkbox" ng-model="todo.completed">
 							<label ng-dblclick="TC.editTodo(todo)">{{todo.title}}</label>
 							<button class="destroy" ng-click="TC.removeTodo($index)"></button>
 						</div>
 						<form ng-submit="TC.doneEditing(todo, $index)">
-							<input class="edit" ng-trim="false" ng-model="todo.title" ng-blur="TC.doneEditing(todo, $index)" todo-escape="TC.revertEditing($index)" todo-focus="todo === TC.editedTodo">
+							<input class="edit" 
+								ng-trim="false" 
+								ng-model="todo.title" 
+								ng-blur="TC.doneEditing(todo, $index)" 
+								ng-keydown="($event.keyCode === TC.ESCAPE_KEY) && TC.revertEditing($index)" 
+								todo-focus="todo === TC.editedTodo">
 						</form>
 					</li>
 				</ul>
@@ -65,6 +71,5 @@
 		<script src="js/controllers/todoCtrl.js"></script>
 		<script src="js/services/todoStorage.js"></script>
 		<script src="js/directives/todoFocus.js"></script>
-		<script src="js/directives/todoEscape.js"></script>
 	</body>
 </html>

examples/angularjs-perf/js/app.js

+/* jshint undef: true, unused: true */
 /*global angular */
-/*jshint unused:false */
 (function () {
 	'use strict';
 
@@ -8,5 +8,5 @@
 	 *
 	 * @type {angular.Module}
 	 */
-	angular.module('todomvc', ['todoCtrl', 'todoEscape', 'todoFocus', 'todoStorage']);
+	angular.module('todomvc', ['todoCtrl', 'todoFocus', 'todoStorage']);
 })();

examples/angularjs-perf/js/controllers/todoCtrl.js

+/* jshint undef: true, unused: true */
 /*global angular */
+
+/*
+ * Line below lets us save `this` as `TC`
+ * to make properties look exactly the same as in the template
+ */
+//jscs:disable safeContextKeyword
 (function () {
 	'use strict';
 
@@ -13,10 +20,15 @@
 		var TC = this;
 		var todos = TC.todos = todoStorage.get();
 
-		TC.newTodo = {title: '', completed: false};
-
+		TC.ESCAPE_KEY = 27;
 		TC.editedTodo = {};
 
+		function resetTodo() {
+			TC.newTodo = {title: '', completed: false};
+		}
+
+		resetTodo();
+
 		if ($location.path() === '') {
 			$location.path('/');
 		}
@@ -43,8 +55,7 @@
 			}
 
 			todos.push(TC.newTodo);
-
-			TC.newTodo = {title: '', completed: false};
+			resetTodo();
 		};
 
 		TC.editTodo = function (todo) {
@@ -64,8 +75,8 @@
 		};
 
 		TC.revertEditing = function (index) {
+			TC.editedTodo = {};
 			todos[index] = TC.originalTodo;
-			TC.doneEditing(TC.originalTodo);
 		};
 
 		TC.removeTodo = function (index) {
@@ -85,3 +96,4 @@
 		};
 	});
 })();
+//jscs:enable

examples/angularjs-perf/js/directives/todoEscape.js

-/*global angular */
-(function () {
-	'use strict';
-
-	angular.module('todoEscape', [])
-
-	/**
-	 * Directive that executes an expression when the element it is applied to gets
-	 * an `escape` keydown event.
-	 */
-	.directive('todoEscape', function () {
-		var ESCAPE_KEY = 27;
-		return function (scope, elem, attrs) {
-			elem.bind('keydown', function (event) {
-				if (event.keyCode === ESCAPE_KEY) {
-					scope.$apply(attrs.todoEscape);
-				}
-			});
-
-			scope.$on('$destroy', function () {
-				elem.unbind('keydown');
-			});
-		};
-	});
-})();

examples/angularjs-perf/js/directives/todoFocus.js

+/* jshint undef: true, unused: true */
 /*global angular */
 (function () {
 	'use strict';

examples/angularjs-perf/js/services/todoStorage.js

+/* jshint undef: true, unused: true */
 /*global angular */
 (function () {
 	'use strict';