Fixing a bug where selecting a node leaks state to the previous tab. I believe the wrong setNode hook is being passed to nodes when switching tabs temporarily. Probably a race condition in the hooks executing. I wasn't able to run that down but found a workaround.
This PR adds some defensive checks to prevent stale hooks as described above. It contains the issue with no apparent side effects.
Changes
[List out the changes you've made in this pull request. Be as specific as possible.]
How Tested
[Explain how you tested this pull request. Include any relevant steps or scripts.]
Description
Fixing a bug where selecting a node leaks state to the previous tab. I believe the wrong
setNodehook is being passed to nodes when switching tabs temporarily. Probably a race condition in the hooks executing. I wasn't able to run that down but found a workaround.This PR adds some defensive checks to prevent stale hooks as described above. It contains the issue with no apparent side effects.
Changes
[List out the changes you've made in this pull request. Be as specific as possible.]
How Tested
[Explain how you tested this pull request. Include any relevant steps or scripts.]
TODOs
Address the root cause of the stale hook