Skip to content
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

Over Eager Rebuilds #1

Closed
slangley opened this issue May 30, 2022 · 4 comments · Fixed by #3
Closed

Over Eager Rebuilds #1

slangley opened this issue May 30, 2022 · 4 comments · Fixed by #3

Comments

@slangley
Copy link
Contributor

When clicking on each tab in sequence (in the example), the tab being clicked on is reloaded (expected), as well as all tabs previously clicked on (unexpected). This results in potentially expensive build() calls to be run every time you click on a tab.

@okaryo
Copy link
Owner

okaryo commented May 31, 2022

@slangley
Thanks for creating the Issue and the PR!

I'll make sure it works here as well and I'll review the code for the PR. Please forgive me if I have to make some minor modifications on my end, such as the style of the code. For example, I may remove debug print.

I have to go to work now, so please wait a bit for me to respond..

@slangley
Copy link
Contributor Author

slangley commented May 31, 2022 via email

@okaryo
Copy link
Owner

okaryo commented Jun 4, 2022

Sorry to keep you waiting.

I reproduced the same problem in my environment. And this is over eager rebuilds as you said.
My knowledge of key and rebuild was poor, so I studied a little.
I will now look at the pull request code you created.

@okaryo okaryo linked a pull request Jun 4, 2022 that will close this issue
@okaryo okaryo linked a pull request Jun 11, 2022 that will close this issue
@okaryo
Copy link
Owner

okaryo commented Jun 11, 2022

fixed by #2 and #3

@okaryo okaryo closed this as completed Jun 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants