From fc7fac22a00c32511f87d8fa815a9bef553f72ea Mon Sep 17 00:00:00 2001
From: Christian Mollekopf <chrigi_1@fastmail.fm>
Date: Thu, 20 Jul 2017 23:52:53 +0200
Subject: Improved key-navigation for treeview

---
 framework/qml/TreeView.qml | 49 ++++++++++++++++++++++++++++++++++++----------
 1 file changed, 39 insertions(+), 10 deletions(-)

(limited to 'framework/qml/TreeView.qml')

diff --git a/framework/qml/TreeView.qml b/framework/qml/TreeView.qml
index 445ad673..6fd3084f 100644
--- a/framework/qml/TreeView.qml
+++ b/framework/qml/TreeView.qml
@@ -74,28 +74,57 @@ Flickable {
             // }
         }
 
+        function selectFirst()
+        {
+            treeView.selection.setCurrentIndex(model.index(0, 0), ItemSelectionModel.ClearAndSelect)
+        }
+
+        function selectNext()
+        {
+            var nextIndex = model.sibling(selection.currentIndex.row + 1, 0, selection.currentIndex)
+            if (nextIndex.valid) {
+                treeView.selection.setCurrentIndex(nextIndex, ItemSelectionModel.ClearAndSelect)
+            } else {
+                var childIndex = model.index(0, 0, selection.currentIndex)
+                if (childIndex.valid) {
+                    if (!treeView.isExpanded(selection.currentIndex)) {
+                        treeView.expand(selection.currentIndex)
+                    }
+                    treeView.selection.setCurrentIndex(childIndex, ItemSelectionModel.ClearAndSelect)
+                }
+            }
+        }
+
+        function selectPrevious()
+        {
+            var previousIndex = model.sibling(selection.currentIndex.row - 1, 0, selection.currentIndex)
+            if (previousIndex.valid) {
+                treeView.selection.setCurrentIndex(previousIndex, ItemSelectionModel.ClearAndSelect)
+            } else {
+                var parentIndex = model.parent(selection.currentIndex)
+                if (parentIndex.valid) {
+                    treeView.selection.setCurrentIndex(parentIndex, ItemSelectionModel.ClearAndSelect)
+                }
+            }
+        }
+
         onActiveFocusChanged: {
             //Set an initially focused item when the list view receives focus
             if (activeFocus && !selection.hasSelection) {
-                treeView.selection.setCurrentIndex(model.index(0, 0), ItemSelectionModel.ClearAndSelect)
+                selectFirst()
             }
         }
 
         Keys.onDownPressed: {
             if (!selection.hasSelection) {
-                treeView.selection.setCurrentIndex(model.index(0, 0), ItemSelectionModel.ClearAndSelect)
+                selectFirst()
             } else {
-                treeView.selection.setCurrentIndex(model.sibling(selection.currentIndex.row + 1, 0, selection.currentIndex), ItemSelectionModel.ClearAndSelect)
+                selectNext();
             }
         }
 
-        Keys.onUpPressed: {
-            treeView.selection.setCurrentIndex(model.sibling(selection.currentIndex.row - 1, 0, selection.currentIndex), ItemSelectionModel.ClearAndSelect)
-        }
-
-        Keys.onReturnPressed: {
-            treeView.activated(selection.currentIndex)
-        }
+        Keys.onUpPressed: selectPrevious()
+        Keys.onReturnPressed: treeView.activated(selection.currentIndex)
 
         //Forward the signal because on a desktopsystem activated is only triggerd by double clicks
         onClicked: treeView.activated(index)
-- 
cgit v1.2.3