Hi Steve,
Thank you for the video and detailed analysis. We have changed the MoveTo so the destination parameter should always be assigned. There was an issue that when passing nil to the destination node that the treeview would lock up as well as moving it to itself, so the MoveTo is actually more strict that it was before. The MoveTo is actually used to move a node inside another child or parent node, whenever the parameter is nil, nothing will happen. The move operation that you want to use is actually basically an index change on collection level so to have the same effect, you can use the following code instead (for the upward movement):
var
i: Integer;
begin
i := ctvColumns.SelectedNode.Index;
ctvColumns.SelectedNode.Index := i - 1;
ctvColumns.SelectedNode := ctvColumns.Nodes.Items[i - 1];