New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

2017-ncueie-isomorphic #1

Open

93i7xo2 wants to merge 1 commit into hellojs-tw:master from 93i7xo2:master

93i7xo2 commented Aug 10, 2017

No description provided.


           finish

02ef5e3

yanhsu suggested changes

View reviewed changes

src/App.jsx

+                removeElement = (idx) => {
+                  //不想用setState所以用forceUpdate
+                  this.state.list.splice(idx,1);

yanhsu Aug 14, 2017

盡量不要用forceupdate喇

Author

93i7xo2 Aug 14, 2017

this.setState({...this.state}) //可是我覺得這樣很沒有freestyle

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet