From 1ca05b0bc78c8f17dda7a4026ab3983a07a25be1 Mon Sep 17 00:00:00 2001
From: Christian Mollekopf <chrigi_1@fastmail.fm>
Date: Wed, 26 Jul 2017 09:59:26 -0600
Subject: TreeView: activeIndex instead of focus to highlight focused index

---
 framework/qml/TreeView.qml | 33 ++++++++++++++++++++++++++-------
 1 file changed, 26 insertions(+), 7 deletions(-)

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

diff --git a/framework/qml/TreeView.qml b/framework/qml/TreeView.qml
index 871a21975..5dc05f3a 100644
--- a/framework/qml/TreeView.qml
+++ b/framework/qml/TreeView.qml
@@ -30,8 +30,16 @@ FocusScope {
     default property alias __columns: treeView.__columns
     property alias model: treeView.model
     property alias currentIndex: treeView.currentIndex
-    signal dropped(QtObject drop, QtObject model)
+    /*
+     * Because active focus is useless in list/treeviews we use the concept of an activeIndex.
+     * The current selection represents the focused index. The activeIndex represents the selected index.
+     */
+    property var activeIndex: null
+    signal dropped(var drop, var model)
     signal activated(var index)
+    onActivated: {
+        activeIndex = index
+    }
 
     Flickable {
         id: flickableItem
@@ -147,16 +155,27 @@ FocusScope {
             alternatingRowColors: false
             headerVisible: false
 
-            //TODO instead of highlighting the current selection: underline the current selection, and highlight the last activated index, so it corresponds
-            //to what we have in the maillist view.
-            //* underline: activefocus
-            //* glow: hover
-            //* highlight: selected
             style: TreeViewStyle {
                 rowDelegate: Rectangle {
-                    color: styleData.selected ? Kube.Colors.highlightColor : Kube.Colors.textColor
+                    //FIXME Uses internal API to get to the model index
+                    property bool isActive: root.activeIndex === treeView.__model.mapRowToModelIndex(styleData.row)
+
                     height: Kube.Units.gridUnit * 1.5
                     width: parent.width
+                    color: Kube.Colors.textColor
+
+                    Rectangle {
+                        anchors.fill: parent
+                        color: Kube.Colors.highlightColor
+                        visible: isActive
+                    }
+                    Rectangle {
+                        anchors.fill: parent
+                        border.width: 2
+                        border.color: Kube.Colors.focusedButtonColor
+                        color: "transparent"
+                        visible: styleData.selected
+                    }
                 }
 
                 frame: Rectangle {
-- 
cgit v1.2.3