Merge bitcoin-core#371: Remember user's peer sort preference

9cd558e qml: remember user's peer sort preference (jarolrod)

Pull request description:

  This sets the sort preference for the peers table in qsettings so that
  it can be remembered. Also addresses an issue where the `id` toggle button is always checked when you go back to the table even though the table is not sorted by `id`.

  [![Build Artifacts](https://img.shields.io/badge/Build%20Artifacts-green
  )](https://github.com/bitcoin-core/gui-qml/actions/runs/6819663186)

ACKs for top commit:
  pablomartin4btc:
    tACK 9cd558e

Tree-SHA512: 18f6a42969f1d5099dcfdbb14bfd366e90977a5aa7c730ab516dc0e4d67687159f6e6daa356a2f2a8111a856932624a772c1f076353ef69c5c707708a9c9df58

src/qml/pages/node/Peers.qml

@@ -5,6 +5,7 @@
 import QtQuick 2.15
 import QtQuick.Controls 2.15
 import QtQuick.Layouts 1.15
+import Qt.labs.settings 1.0
 import org.bitcoincore.qt 1.0
 import "../../controls"
 import "../../components"
@@ -28,6 +29,15 @@ Page {
         }
     }
 
+    Settings {
+        id: settings
+        property string peerListSortBy: "nodeId"
+    }
+
+    Component.onCompleted: {
+        peerListModelProxy.sortBy = settings.peerListSortBy
+    }
+
     ListView {
         id: listView
         clip: true
@@ -64,44 +74,55 @@ Page {
                     ToggleButton {
                         text: qsTr("ID")
                         autoExclusive: true
-                        checked: true
+                        checked: settings.peerListSortBy === "nodeId"
                         onClicked: {
                             peerListModelProxy.sortBy = "nodeId"
+                            settings.peerListSortBy = "nodeId"
                         }
                     }
                     ToggleButton {
                         text: qsTr("Direction")
                         autoExclusive: true
+                        checked: settings.peerListSortBy === "direction"
                         onClicked: {
                             peerListModelProxy.sortBy = "direction"
+                            settings.peerListSortBy = "direction"
                         }
                     }
                     ToggleButton {
                         text: qsTr("User Agent")
                         autoExclusive: true
+                        checked: settings.peerListSortBy === "subversion"
                         onClicked: {
                             peerListModelProxy.sortBy = "subversion"
+                            settings.peerListSortBy = "subversion"
                         }
                     }
                     ToggleButton {
                         text: qsTr("Type")
                         autoExclusive: true
+                        checked: settings.peerListSortBy === "connectionType"
                         onClicked: {
                             peerListModelProxy.sortBy = "connectionType"
+                            settings.peerListSortBy = "connectionType"
                         }
                     }
                     ToggleButton {
                         text: qsTr("Ip")
                         autoExclusive: true
+                        checked: settings.peerListSortBy === "address"
                         onClicked: {
                             peerListModelProxy.sortBy = "address"
+                            settings.peerListSortBy = "address"
                         }
                     }
                     ToggleButton {
                         text: qsTr("Network")
                         autoExclusive: true
+                        checked: settings.peerListSortBy === "network"
                         onClicked: {
                             peerListModelProxy.sortBy = "network"
+                            settings.peerListSortBy = "network"
                         }
                     }
                 }