Merge pull request #56 from timotius02/master

fix bind warning in todoApp example
This commit is contained in:
Jacob Wenger
2015-08-24 10:54:07 -07:00
2 changed files with 2 additions and 4 deletions

View File

@@ -6,7 +6,7 @@ var TodoList2 = React.createClass({
return (
<li key={ index }>
{ item.text }
<span onClick={ _this.props.removeItem.bind(_this, item['.key']) }
<span onClick={ _this.props.removeItem.bind(null, item['.key']) }
style={{ color: 'red', marginLeft: '10px', cursor: 'pointer' }}>
X
</span>
@@ -50,7 +50,6 @@ var TodoApp2 = React.createClass({
},
removeItem: function(key) {
console.log('removeItem()', key);
var firebaseRef = new Firebase('https://ReactFireTodoApp.firebaseio.com/items/');
firebaseRef.child(key).remove();
},

View File

@@ -6,7 +6,7 @@ var TodoList3 = React.createClass({
return (
<li key={ index }>
{ item.text }
<span onClick={ _this.props.removeItem.bind(_this, item['.key']) }
<span onClick={ _this.props.removeItem.bind(null, item['.key']) }
style={{ color: 'red', marginLeft: '10px', cursor: 'pointer' }}>
X
</span>
@@ -37,7 +37,6 @@ var TodoApp3 = React.createClass({
},
removeItem: function(key) {
console.log('removeItem()', key);
var firebaseRef = new Firebase('https://ReactFireTodoApp.firebaseio.com/items/');
firebaseRef.child(key).remove();
},