-
Notifications
You must be signed in to change notification settings - Fork 78
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
Fix the rerender loop of profile hero data #985
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/metafam/the-game/FkrtrC9EhfrELWHQoYJ7G4G3rr1i |
Hi @vidvidvid , I fixed that UI issue in my #962 issue, this was a broad issue that I created and I sent my commits to that branch. Maybe we can deploy that issue. You can check my fix/several-ui-improvements branch. The problem was happening because useAnimation is having updateFN as a dependant and that was creating an infinite loop, so just removing that was successful.
|
aaaa, okay, i see. will fix my pr than to match your solution and test if the rest of the components where i used |
The rest of the components were subscribed to the same |
LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Overview
What features/fixes does this PR include?
As the title suggests + refactored the other player profile sections' animations with the custom
useAnimation
hook.Renamed
useAnimation
touseAnimateProfileChanges
.Please provide the GitHub issue number
Closes #984
Follow up Improvement Ideas
useAnimation
hook to something else to avoid the confusion (https://usehooks.com/useAnimation/)Implementation
Describe technical (nontrivial / non-obvious) parts of your code
Read the comment
paired-with: @firattale