Location selection more responsive

5 files changed, 93 insertions, 42 deletions

diff --git a/gui/backend.go b/gui/backend.go
index 064bd94..f693118 100644
--- a/gui/backend.go
+++ b/gui/backend.go
@@ -35,6 +35,11 @@ func UseLocation(label string) {
 	backend.UseLocation(label)
 }
 
+//export UseAutomaticGateway
+func UseAutomaticGateway() {
+	backend.UseAutomaticGateway()
+}
+
 //export UseTransport
 func UseTransport(transport string) {
 	backend.UseTransport(transport)
diff --git a/gui/handlers.cpp b/gui/handlers.cpp
index 558c288..9f68834 100644
--- a/gui/handlers.cpp
+++ b/gui/handlers.cpp
@@ -47,6 +47,11 @@ void Backend::useLocation(QString label)
     UseLocation(toGoStr(label));
 }
 
+void Backend::useAutomaticGateway()
+{
+    UseAutomaticGateway();
+}
+
 void Backend::login(QString username, QString password)
 {
     Login(toGoStr(username), toGoStr(password));
diff --git a/gui/handlers.h b/gui/handlers.h
index 9d949fc..3415b7e 100644
--- a/gui/handlers.h
+++ b/gui/handlers.h
@@ -37,6 +37,7 @@ public slots:
     void donateAccepted();
     void donateSeen();
     void useLocation(QString username);
+    void useAutomaticGateway();
     void login(QString username, QString password);
     void resetError(QString errlabel);
     void resetNotification(QString label);
diff --git a/gui/qml/VpnState.qml b/gui/qml/VpnState.qml
index 886868f..7ec6695 100644
--- a/gui/qml/VpnState.qml
+++ b/gui/qml/VpnState.qml
@@ -25,6 +25,10 @@ StateGroup {
                 text: toHuman("off")
             }
             PropertyChanges {
+                target: autoSelectionItem
+		text: qsTr("Best")
+            }
+            PropertyChanges {
                 target: mainStatus
                 text: toHuman("off")
             }
@@ -46,7 +50,18 @@ StateGroup {
             }
             PropertyChanges {
                 target: statusItem
-                text: toHumanWithLocation("on")
+                text: toHuman("on")
+            }
+            PropertyChanges {
+                target: autoSelectionItem
+		text: {
+			if (autoSelectionButton.checked) {
+				//: %1 -> location to which the client is connected to
+				qsTr("Best (%1)").arg(locationStr())
+			} else {
+				qsTr("Best")
+			}
+		}
             }
             PropertyChanges {
                 target: mainStatus
@@ -54,7 +69,8 @@ StateGroup {
             }
             PropertyChanges {
                 target: mainCurrentGateway
-                text: qsTr("Connected to ") + ctx.currentLocation
+		//: %1 -> location to which the client is connected to
+                text: qsTr("Connected to %1").arg(locationStr())
             }
         },
         State {
@@ -66,7 +82,18 @@ StateGroup {
             }
             PropertyChanges {
                 target: statusItem
-                text: toHumanWithLocation("connecting")
+                text: toHuman("connecting")
+            }
+            PropertyChanges {
+                target: autoSelectionItem
+		text: {
+			if (autoSelectionButton.checked) {
+				//: %1 -> location to which the client is connected to
+				qsTr("Best (%1)").arg(locationStr())
+			} else {
+				qsTr("Best")
+			}
+		}
             }
             PropertyChanges {
                 target: mainStatus
@@ -89,6 +116,10 @@ StateGroup {
                 text: toHuman("stopping")
             }
             PropertyChanges {
+                target: autoSelectionItem
+		text: qsTr("Best")
+            }
+            PropertyChanges {
                 target: mainStatus
                 text: toHuman("stopping")
             }
@@ -109,6 +140,10 @@ StateGroup {
                 text: toHuman("failed")
             }
             PropertyChanges {
+                target: autoSelectionItem
+		text: qsTr("Best")
+            }
+            PropertyChanges {
                 target: mainStatus
                 text: toHuman("failed")
             }
diff --git a/gui/qml/main.qml b/gui/qml/main.qml
index 211dbe5..ddd049d 100644
--- a/gui/qml/main.qml
+++ b/gui/qml/main.qml
@@ -32,7 +32,7 @@ Window {
             text: qsTr("Info")
         }
         TabButton {
-            text: qsTr("Gateways")
+            text: qsTr("Location")
         }
     }
 
@@ -128,7 +128,7 @@ Window {
                     color: "grey"
                     text: qsTr("Location has been manually set.")
                     anchors.horizontalCenter: parent.horizontalCenter
-                    visible: manualSelectionButton.checked
+                    visible: isManualLocation()
                 }
             }
         }
@@ -145,14 +145,15 @@ Window {
 
                 RadioButton {
                     id: autoSelectionButton
-                    checked: true
-                    text: qsTr("Automatic")
-                    // TODO still needs to change to automatic on the backend, and maybe note
-                    // that the change will be effective on the next reconnect.
+                    checked: !isManualLocation()
+                    text: qsTr("Best")
+		    onClicked: backend.useAutomaticGateway()
                 }
                 RadioButton {
                     id: manualSelectionButton
+                    checked: isManualLocation()
                     text: qsTr("Manual")
+		    onClicked: setGwSelection()
                 }
                 ComboBox {
                     id: gwSelector
@@ -160,7 +161,7 @@ Window {
                     visible: manualSelectionButton.checked
                     anchors.horizontalCenter: parent.horizontalCenter
 
-                    model: [qsTr("Automatic")]
+                    model: [qsTr("Best")]
                     onActivated: {
                         console.debug("Selected gateway:", currentText)
                         backend.useLocation(currentText.toString())
@@ -249,6 +250,24 @@ Window {
         return false
     }
 
+    function isManualLocation() {
+	if (!ctx) {
+            return false
+	}
+	return ctx.manualLocation == "true"
+    }
+
+    function setGwSelection() {
+	if (!ctx.currentLocation) {
+		return
+	}
+
+	const location = ctx.currentLocation.toLowerCase()
+	const idx = gwSelector.model.indexOf(location)
+	gwSelector.currentIndex = idx
+	backend.useLocation(location)
+    }
+
     Component.onCompleted: {
         loginDone = false
         console.debug("Platform:", Qt.platform.os)
@@ -282,30 +301,8 @@ Window {
         }
     }
 
-    /* TODO change this!!! automatic: Paris (FR) */
-
-    function toHumanWithLocation(st) {
-        switch (st) {
-        case "off":
-            //: %1 -> application name
-            return qsTr("%1 off").arg(ctx.appName)
-        case "on":
-            //: %1 -> application name
-            //: %2 -> current gateway
-            return qsTr("%1 on - %2").arg(ctx.appName).arg(ctx.currentLocation)
-        case "connecting":
-            //: %1 -> application name
-            //: %2 -> current gateway
-            return qsTr("Connecting to %1 - %2").arg(ctx.appName).arg(
-                        ctx.currentLocation)
-        case "stopping":
-            //: %1 -> application name
-            return qsTr("Stopping %1").arg(ctx.appName)
-        case "failed":
-            //: %1 -> application name
-            return qsTr("%1 blocking internet").arg(
-                        ctx.appName) // TODO failed is not handed yet
-        }
+    function locationStr() {
+	    return ctx.currentLocation + ", " + ctx.currentCountry
     }
 
     property var icons: {
@@ -355,26 +352,34 @@ Window {
                 enabled: false
             }
 
+            MenuSeparator {}
+
             MenuItem {
                 id: autoSelectionItem
-                text: qsTr("automatic")
+                text: qsTr("Best")
                 checkable: true
-                checked: true
-                enabled: false
+                checked: !isManualLocation()
+                onTriggered: {
+                	backend.useAutomaticGateway()
+                }
             }
 
 
             /* a minimal segfault for submenu */
             // Menu {}
 
-            MenuSeparator {}
-
             MenuItem {
                 id: manualSelectionItem
-                text: qsTr("Pick gateway…")
+		text: {
+			if (isManualLocation()) {
+				locationStr()
+			} else {
+				qsTr("Pick location…")
+			}
+		}
                 checkable: true
-                checked: false
-                enabled: true
+                checked: isManualLocation()
+                onTriggered: setGwSelection()
             }
 
             MenuSeparator {}