Vanillajs app: Fix bug when removing a single todo item

124b1c99 · Sindre Sorhus · 8d004d8f · 124b1c99
Commit 124b1c99 authored Jun 24, 2012 by Sindre Sorhus
Show whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

vanilla-examples/vanillajs/js/app.js vanilla-examples/vanillajs/js/app.js +2 -2

No files found.
--- a/vanilla-examples/vanillajs/js/app.js
+++ b/vanilla-examples/vanillajs/js/app.js
@@ -121,9 +121,9 @@ function editTodo( todoId, text ) {
 }

 function removeTodoById( id ) {
-	var i, l;
+	var i = todos.length;

-	for ( i = 0, l = todos.length; i < l; i++ ) {
+	while ( i-- ) {
 		if ( todos[ i ].id === id ) {
 			todos.splice( i, 1 );
 		}