QML UI: Remember episode list scroll position (bug 1660)

When deleting episodes, the vertical scroll position should not be reset.
gpodder · Sep 5, 2012 · a14a7eb · a14a7eb
1 parent a216613
commit a14a7eb
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 0 deletions.
diff --git a/share/gpodder/ui/qml/EpisodeList.qml b/share/gpodder/ui/qml/EpisodeList.qml
@@ -8,6 +8,12 @@ import 'config.js' as Config
 Item {
     id: episodeList
     property string currentFilterText
+    property string mainState
+
+    onMainStateChanged: {
+        // Don't remember contentY when leaving episode list
+        listView.lastContentY = 0;
+    }
 
     property alias model: listView.model
     property alias moving: listView.moving
@@ -51,6 +57,23 @@ Item {
             }
         }
 
+        property real lastContentY: 0
+
+        onContentYChanged: {
+            // Keep Y scroll position when deleting episodes (bug 1660)
+            if (contentY === 0) {
+                if (lastContentY > 0) {
+                    contentY = lastContentY;
+                }
+            } else {
+                if (episodeList.mainState === 'episodes') {
+                    // Only store scroll position when the episode list is
+                    // shown (avoids overwriting it in onMainStateChanged)
+                    lastContentY = contentY;
+                }
+            }
+        }
+
         anchors.fill: parent
         property int openedIndex: -1
         visible: count > 0

diff --git a/share/gpodder/ui/qml/Main.qml b/share/gpodder/ui/qml/Main.qml
@@ -207,6 +207,7 @@ Image {
 
         EpisodeList {
             id: episodeList
+            mainState: main.state
 
             opacity: 0