Merge pull request #66 from Xmasjos/feature/allReadRefreshAllSetting

[Add] Setting for new top menu item `All read & Refresh all`
poetaster · Jun 4, 2021 · 938e5e9 · 938e5e9
2 parents 8be9db9 + ba9899c
commit 938e5e9
Show file tree

Hide file tree

Showing 3 changed files with 25 additions and 0 deletions.
diff --git a/qml/harbour-tidings.qml b/qml/harbour-tidings.qml
@@ -95,6 +95,12 @@ ApplicationWindow
         value: "1"
     }
 
+    ConfigValue {
+        id: configEnableRefreshCombined
+        key: "menu-enable-refresh-combined"
+        value: "1"
+    }
+
     ConfigValue {
         id: configLoadImages
         key: "view-load-images"

diff --git a/qml/pages/SettingsPage.qml b/qml/pages/SettingsPage.qml
@@ -40,6 +40,24 @@ Page {
                 height: Theme.paddingLarge
             }
 
+            TextSwitch {
+                width: parent.width
+                text: qsTr("Enable 'All read & Refresh all'")
+                description: qsTr("Whether to show or hide the top menu item for 'All read & Refresh all'")
+                automaticCheck: false
+                checked: configEnableRefreshCombined.booleanValue
+
+                onClicked: {
+                    configEnableRefreshCombined.value =
+                            configEnableRefreshCombined.booleanValue ? "0" : "1";
+                }
+            }
+
+            Item {
+                width: 1
+                height: Theme.paddingLarge
+            }
+
             TextSwitch {
                 width: parent.width
                 text: qsTr("Load images automatically")

diff --git a/qml/pages/SourcesPage.qml b/qml/pages/SourcesPage.qml
@@ -249,6 +249,7 @@ Page {
                 }
             }
             MenuItem {
+                visible: configEnableRefreshCombined.booleanValue
                 text: newsBlendModel.busy ? qsTr("Abort refreshing")
                                           : qsTr("All read & Refresh all")