Merge pull request #31 from spicyj/immutable-state

Change TodoMVC to not modify state in place
facebook · May 31, 2013 · bb4788e · bb4788e
2 parents b20a7c2 + 6e805dd
commit bb4788e
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/examples/todomvc/js/app.js b/examples/todomvc/js/app.js
@@ -98,12 +98,12 @@ var TodoApp = React.createClass({
     var val = this.refs.newField.getDOMNode().value.trim();
     if (val) {
       var todos = this.state.todos;
-      todos.push({
+      var newTodo = {
         id: Utils.uuid(),
         title: val,
         completed: false
-      });
-      this.setState({todos: todos});
+      };
+      this.setState({todos: todos.concat([newTodo])});
       this.refs.newField.getDOMNode().value = '';
     }
     return false;