-
Notifications
You must be signed in to change notification settings - Fork 664
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.
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
05.extra-1.js all the cells render when you click one of them #108
Comments
I'm afraid I don't have time to answer this in the issues, but you're welcome to bring questions like this to my office hours: https://kcd.im/office-hours. |
I shouldn't have asked the question here because it is not the main subject in this issue, so forget about it 😄 Your solution to this exercise is not accurate and does not work as expected (all the cells render). You my also need to re-record the video for this. |
I'm pretty confident I've looked at this before and you're misunderstanding something. I don't have time to look into it right now but I can during office hours if you wanna ask there. |
For anyone that noticed this issue, it will be reviewed in exercise 6.2. Exercise 5.1 only focuses on the re-rendering of the |
Even with splitting the contexts, all the cells still render. This is also visible on the production build on Netlify
https://react-performance.netlify.app/isolated/final/05.extra-1.js
And this is logicial because when you click on a cell the state (context) changes for all the cells.
Is there any other solution to avoid rendering all the cells?
The text was updated successfully, but these errors were encountered: