Retain expanded nodes

[JogoShugh]

As a developer, I want to be able to add new secondary workitems quickly so that I can write tasks and tests in my IDE as they occur to me.

The client is forcing a full list refresh, including flashing the window white and collapsing any selected nodes when the user clicks the add Defect, Task, or Test buttons. This is very painful behavior, and makes the experience very poor.

How to Reproduce

• Expand a Story node
• Select that node
• Click the add Task button in the button bar
• Current Behavior:
  • The Tasks window flashes white
  • It displays a waiting cursor
  • It collapses the currently selected nodes
  • It adds the new item to the bottom of the previously selected node (which is invisible until you expand the node again).
• Repeat the process with Test.

  Acceptance Criteria

• When executing the steps above, the client should retain the expanded nodes
• Nice to have: not flash the window blank.

[MatiasHeffel]

We've found that when something change in the model of the treeview everything in the tree is refreshed and collapsed. Looking the documentation of this library we found the following issue with this patch: http://sourceforge.net/p/treeviewadv/patches/10/

On the other hand, looking in the Sample code in this library, there is an example where a node is added using the parent of the node as argument

 OnStructureChanged(new TreePathEventArgs(parent));

The plugin is using another implementation, so we can try two approaches:

• One: is try to change the code and the implementation to use this method OnStructureChanged (using parent as argument).
• Two: is replicate the patch and save the nodes that are expanded and restored after the tree collapsed.

We are current work in both alternatives and then discuss which is the best solution.

[davidhaimovich]

Solution number 1 worked well.

Replacing: model.InvokeStructureChanged();

to model.InvokeStructureChanged(treePath);

Makes the tree refresh only the section inside treePath node and not all the tree. This will allow us to avoid collapsing all the tree, after adding an asset.

[MatiasHeffel]

Revert Item when it was just created a new node it not collapsing the treeview anymore, using the same approach that we used before.

[MatiasHeffel]

To DO

• Check and Fix the Test
• Check the performance improvements
• Check other functionality in the context menu is still working.