more fix

gui/components/MainView.qml

@@ -77,6 +77,9 @@ Page {
             text: qsTr("Quit")
             icon: "../resources/quit.svg"
             triggered: function() {
+                if (ctx.status == "on") {
+                    backend.switchOff()
+                }
                 Qt.callLater(backend.quit)
             }
         }

gui/components/StatusBox.qml

@@ -56,13 +56,6 @@ Item {
         }
         onClicked: {
             settingsDrawer.toggle()
-            /*
-            if (stackView.depth > 1) {
-                stackView.pop()
-            } else {
-                settingsDrawer.toggle()
-            }
-            */
         }
 
         Icon {
@@ -71,10 +64,6 @@ Item {
             height: 16
             anchors.centerIn: settingsButton
             source: "../resources/gear-fill.svg"
-            /*
-            source: stackView.depth
-                    > 1 ? "../resources/arrow-left.svg" : "../resources/gear-fill.svg"
-            */
         }
     }
 

gui/components/Systray.qml

@@ -55,7 +55,12 @@ Labs.SystemTrayIcon {
         Labs.MenuItem {
             //: Part of the systray menu; quits the application
             text: qsTr("Quit")
-            onTriggered: backend.quit()
+            onTriggered: {
+                if (ctx.status == "on") {
+                    backend.switchOff()
+                }
+                backend.quit()
+            }
         }
     }
 

pkg/helper/darwin.go

@@ -36,6 +36,7 @@ import (
 	"path/filepath"
 	"strconv"
 	"strings"
+	"time"
 
 	"github.com/sevlyar/go-daemon"
 )
@@ -136,7 +137,7 @@ func firewallStop() error {
 		/* TODO return error if different from anchor not exists */
 		/*return errors.New("Error while stopping firewall")*/
 	}
-	for i := range [50]int{} {
+	for _ = range [50]int{} {
 		if firewallIsUp() {
 			log.Printf("Firewall still up, waiting...")
 			time.Sleep(200 * time.Millisecond)