dirlist/dirtree: context handling

Add a NewContext() function that extracts and handles the canceling and
re-creation of a new context for the directory list. This enables a
better control of the context cancellation especially for a directory
tree when a virtual node is selected. Before, the previous folder would
have been opened if a virtual node was selected.

Signed-off-by: Koni Marti <koni.marti@gmail.com>
Acked-by: Robin Jarry <robin@jarry.cc>

1 files changed, 7 insertions, 5 deletions

diff --git a/app/dirtree.go b/app/dirtree.go
index 6a3b34c0..8f8730b4 100644
--- a/app/dirtree.go
+++ b/app/dirtree.go
@@ -256,12 +256,17 @@ func (dt *DirectoryTree) NextPrev(delta int) {
 		}
 	}
 
-	dt.listIdx = newIdx
+	dt.selectIndex(newIdx)
+}
+
+func (dt *DirectoryTree) selectIndex(i int) {
+	dt.listIdx = i
 	if path := dt.getDirectory(dt.list[dt.listIdx]); path != "" {
 		dt.virtual = false
 		dt.Select(path)
 	} else {
 		dt.virtual = true
+		dt.NewContext()
 		dt.virtualCb()
 	}
 }
@@ -274,10 +279,7 @@ func (dt *DirectoryTree) CollapseFolder() {
 				// highlight parent node and select it
 				for i, t := range dt.list {
 					if t == node.Parent {
-						dt.listIdx = i
-						if path := dt.getDirectory(dt.list[dt.listIdx]); path != "" {
-							dt.Select(path)
-						}
+						dt.selectIndex(i)
 					}
 				}
 			} else {