From 51018b7788a87658fd5fccb645a8e808e34d9450 Mon Sep 17 00:00:00 2001
From: Christian Mollekopf <chrigi_1@fastmail.fm>
Date: Wed, 22 Mar 2017 22:01:59 +0100
Subject: Mark the currently viewed mail as read.

The listview is absolutely useless with non-uniform entries.
We have to do or own tracking of the current index, we have to do our
own scrolling, the flickable scrolling is horrible with a mouse,
and even though caching all delegates somewhat improves the situation,
it's still laggy every now and then.

However, this now at least works somewhat correctly.
But we'll really have to look for a new solution for this eventually.
---
 .../package/contents/ui/ConversationView.qml       | 115 +++++++++++++++++++--
 1 file changed, 109 insertions(+), 6 deletions(-)

(limited to 'components/package')

diff --git a/components/package/contents/ui/ConversationView.qml b/components/package/contents/ui/ConversationView.qml
index 3c76928f..6ecc59ae 100644
--- a/components/package/contents/ui/ConversationView.qml
+++ b/components/package/contents/ui/ConversationView.qml
@@ -31,11 +31,43 @@ Rectangle {
     id: root
 
     property variant mail;
+    property int currentIndex: 0;
+    property bool scrollToEnd: true;
+    property variant currentMail: null;
+    onCurrentIndexChanged: {
+        markAsReadTimer.restart();
+    }
+    onMailChanged: {
+        scrollToEnd = true;
+        currentMail = null;
+    }
 
     color: Kirigami.Theme.backgroundColor
 
     ListView {
         id: listView
+        function setCurrentIndex()
+        {
+            /**
+             * This will detect the index at the "scrollbar-position" (visibleArea.yPosition).
+             * This ensures that the first and last entry can become the currentIndex,
+             * but in the middle of the list the item in the middle is set as the current item.
+             */
+            var yPos = 0.5;
+            if (listView.visibleArea.yPosition < 0.4) {
+                yPos = 0.2 + (0.2 * listView.visibleArea.yPosition);
+            }
+            if (listView.visibleArea.yPosition > 0.6) {
+                yPos = 0.6 + (0.2 * listView.visibleArea.yPosition)
+            }
+            var indexAtCenter = listView.indexAt(root.width / 2, contentY + root.height * yPos);
+            if (indexAtCenter >= 0) {
+                root.currentIndex = indexAtCenter;
+            } else {
+                root.currentIndex = count - 1;
+            }
+        }
+
         anchors {
             top: parent.top
             left: parent.left
@@ -64,13 +96,68 @@ Rectangle {
 
         boundsBehavior: Flickable.StopAtBounds
 
-        //Always scroll to the end of the conversation
-        highlightFollowsCurrentItem: true
-        //Scroll quickly
-        highlightMoveDuration: 1
+        //default is 1500, which is not usable with a mouse
+        flickDeceleration: 10000
+
+        //Optimize for view quality
+        pixelAligned: true
+
+        Timer {
+            id: scrollToEndTimer
+            interval: 10
+            running: false
+            repeat: false
+            onTriggered: {
+                //Only do this once per conversation
+                root.scrollToEnd = false;
+                root.currentIndex = listView.count - 1
+                //positionViewAtEnd/Index don't work
+                listView.contentY = listView.contentHeight - listView.height
+            }
+        }
+
         onCountChanged: {
-            //TODO: ideally we should only do this initially, not when new messages enter while you're reading.
-            currentIndex = count - 1;
+            if (root.scrollToEnd) {
+                scrollToEndTimer.restart()
+            }
+        }
+
+        onContentHeightChanged: {
+            //Initially it will resize a lot, so we keep waiting
+            if (root.scrollToEnd) {
+                scrollToEndTimer.restart()
+            }
+        }
+
+        onContentYChanged: {
+            //We have to track our current mail manually
+            setCurrentIndex();
+        }
+
+        //The cacheBuffer needs to be large enough to fit the whole thread.
+        //Otherwise the contentHeight will constantly increase and decrease,
+        //which will break lot's of things.
+        cacheBuffer: 100000
+
+        KubeFramework.MailController {
+            id: mailController
+            Binding on mail {
+                //!! checks for the availability of the type
+                when: !!root.currentMail
+                value: root.currentMail
+            }
+        }
+
+        Timer {
+            id: markAsReadTimer
+            interval: 2000
+            running: false
+            repeat: false
+            onTriggered: {
+                if (mailController.markAsReadAction.enabled) {
+                    mailController.markAsReadAction.execute();
+                }
+            }
         }
 
         //Intercept all scroll events,
@@ -85,6 +172,13 @@ Rectangle {
         id: mailDelegate
 
         Item {
+            id: wrapper
+            property bool isCurrent: root.currentIndex === index;
+            onIsCurrentChanged: {
+                if (isCurrent) {
+                    root.currentMail = model.mail
+                }
+            }
 
             height: sheet.height + Kirigami.Units.gridUnit
             width: parent.width
@@ -95,6 +189,15 @@ Rectangle {
                 implicitHeight: header.height + attachments.height + body.height + footer.height + Kirigami.Units.largeSpacing
                 width: parent.width - Kirigami.Units.gridUnit * 2
 
+                //Overlay for non-active mails
+                Rectangle {
+                    anchors.fill: parent
+                    visible: !wrapper.isCurrent
+                    color: "lightGrey"
+                    z: 1
+                    opacity: 0.2
+                }
+
                 color: Kirigami.Theme.viewBackgroundColor
 
                 //BEGIN header
-- 
cgit v1.2.3