Skip to content
This repository has been archived by the owner on Feb 7, 2023. It is now read-only.

Fix memory leak with the <EasterEggUsername/> #39

Open
zbreit opened this issue Jul 13, 2020 · 0 comments
Open

Fix memory leak with the <EasterEggUsername/> #39

zbreit opened this issue Jul 13, 2020 · 0 comments
Labels
bug Something isn't working refactoring Any code improvements or suggestions

Comments

@zbreit
Copy link
Collaborator

zbreit commented Jul 13, 2020

When you log out while the <EasterEggUsername/> is active, there is a memory leak because the colors are trying to toggle while the component is unmounted.

Doing this will probably involve using the useIsFocused() hook and wrapping the setState() calls w/ if (isFocused) {...}. That will require this component to be refactored into a modular component.

@zbreit zbreit added bug Something isn't working refactoring Any code improvements or suggestions labels Jul 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working refactoring Any code improvements or suggestions
Projects
None yet
Development

No branches or pull requests

1 participant