search

survivejs / react-book

From apprentice to master (CC BY-NC-ND)
https://survivejs.com/react/introduction/
2.02k stars 366 forks source link

Improve State change communication #350

Closed eMerzh closed 8 years ago

eMerzh commented 8 years ago

it might be me but i have a couple of request for the section "Communicating Note State Changes". First, it might be a good idea to describe that before this point, the onblur does not work (as you ask to "edit" to test the changes).

then, i think that the logic about why this pattern is used, must really be more explained. i didn't catch why we need set the state from Note => Notes => App... and why the setState on Note only doesn't work...is it only because App is the root component? or smth else ?

bebraw commented 8 years ago

Thanks for pointing this out! I'll clarify the section.