You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
A large improvement that can be made currently with the TVP implementation is that when a build target is compiled, the root node for that target is replaced. This comes in via the metals/treeViewDidChange. Now even if you have a bunch of stuff underneath that node expanded and nothing really changes in the nodes, the entire node chain is replaced.
Instead we should take a snapshot of that node and everything underneath it. We'll then need to do the retrieval again and then compare the trees to see what needs to be updated instead of just replacing the whole thing.
What this currently means is that if you have the tree open and expanded and you save, the tree will collapse since everything underneath was replaced.
Search terms
TVP caching.
The text was updated successfully, but these errors were encountered:
Task
A large improvement that can be made currently with the TVP implementation is that when a build target is compiled, the root node for that target is replaced. This comes in via the
metals/treeViewDidChange
. Now even if you have a bunch of stuff underneath that node expanded and nothing really changes in the nodes, the entire node chain is replaced.Instead we should take a snapshot of that node and everything underneath it. We'll then need to do the retrieval again and then compare the trees to see what needs to be updated instead of just replacing the whole thing.
What this currently means is that if you have the tree open and expanded and you save, the tree will collapse since everything underneath was replaced.
Search terms
TVP caching.
The text was updated successfully, but these errors were encountered: