Use StackView activation callbacks to perform init and cleanup since visibility callbacks aren't guaranteed to run in any order

app/gui/AppView.qml

@@ -20,36 +20,28 @@ GridView {
     anchors.bottomMargin: 5
     cellWidth: 225; cellHeight: 385;
 
-    // The StackView will trigger a visibility change when
-    // we're pushed onto it, causing our onVisibleChanged
-    // routine to run, but only if we start as invisible
-    visible: false
-
     function computerLost()
     {
         // Go back to the PC view on PC loss
         stackView.pop()
     }
 
-    Component.onCompleted: {
-        // Don't show any highlighted item until interacting with them
-        currentIndex = -1
-    }
-
     SdlGamepadKeyNavigation {
         id: gamepadKeyNav
     }
 
-    onVisibleChanged: {
-        if (visible) {
+    StackView.onActivated: {
         appModel.computerLost.connect(computerLost)
         gamepadKeyNav.enable()
+
+        // Don't show any highlighted item until interacting with them
+        currentIndex = -1
     }
-        else {
+
+    StackView.onDeactivating: {
         appModel.computerLost.disconnect(computerLost)
         gamepadKeyNav.disable()
     }
-    }
 
     function createModel()
     {

app/gui/CliQuitStreamSegue.qml

@@ -21,13 +21,8 @@ Item {
         errorDialog.open()
     }
 
-    // The StackView will trigger a visibility change when
-    // we're pushed onto it, causing our onVisibleChanged
-    // routine to run, but only if we start as invisible
-    visible: false
-
-    onVisibleChanged: {
-        if (visible && !launcher.isExecuted()) {
+    StackView.onActivated: {
+        if (!launcher.isExecuted()) {
             toolBar.visible = false
             launcher.searchingComputer.connect(onSearchingComputer)
             launcher.quittingApp.connect(onQuittingApp)

app/gui/CliStartStreamSegue.qml

@@ -6,8 +6,6 @@ import QtQuick.Dialogs 1.2
 import ComputerManager 1.0
 
 Item {
-    visible: false
-
     function onSearchingComputer() {
         stageLabel.text = "Establishing connection to PC..."
     }
@@ -36,8 +34,8 @@ Item {
         quitAppDialog.open()
     }
 
-    onVisibleChanged: {
-        if (visible && !launcher.isExecuted()) {
+    StackView.onActivated: {
+        if (!launcher.isExecuted()) {
             toolBar.visible = false
             launcher.searchingComputer.connect(onSearchingComputer)
             launcher.searchingApp.connect(onSearchingApp)

app/gui/PcView.qml

@@ -24,11 +24,6 @@ GridView {
     cellWidth: 350; cellHeight: 350;
     objectName: "Computers"
 
-    // The StackView will trigger a visibility change when
-    // we're pushed onto it, causing our onVisibleChanged
-    // routine to run, but only if we start as invisible
-    visible: false
-
     StreamingPreferences {
         id: prefs
     }
@@ -37,16 +32,9 @@ GridView {
         id: gamepadKeyNav
     }
 
-    onVisibleChanged: {
-        if (visible) {
+    StackView.onActivated: {
         gamepadKeyNav.enable()
-        }
-        else {
-            gamepadKeyNav.disable()
-        }
-    }
 
-    Component.onCompleted: {
         // Setup signals on CM
         ComputerManager.computerAddCompleted.connect(addComplete)
 
@@ -54,6 +42,11 @@ GridView {
         currentIndex = -1
     }
 
+    StackView.onDeactivating: {
+        gamepadKeyNav.disable()
+        ComputerManager.computerAddCompleted.disconnect(addComplete)
+    }
+
     function pairingComplete(error)
     {
         // Close the PIN dialog

app/gui/QuitSegue.qml

@@ -14,11 +14,6 @@ Item {
 
     anchors.fill: parent
 
-    // The StackView will trigger a visibility change when
-    // we're pushed onto it, causing our onVisibleChanged
-    // routine to run, but only if we start as invisible
-    visible: false
-
     function quitAppCompleted(error)
     {
         // Display a failed dialog if we got an error
@@ -42,19 +37,18 @@ Item {
         }
     }
 
-    onVisibleChanged: {
-        if (visible) {
+    StackView.onActivated: {
         // Hide the toolbar before we start loading
         toolBar.visible = false
 
         // Connect the quit completion signal
         ComputerManager.quitAppCompleted.connect(quitAppCompleted)
     }
-        else {
+
+    StackView.onDeactivating: {
         // Disconnect the signal
         ComputerManager.quitAppCompleted.disconnect(quitAppCompleted)
     }
-    }
 
     Row {
         anchors.centerIn: parent

app/gui/SettingsView.qml

@@ -28,26 +28,17 @@ Flickable {
         id: prefs
     }
 
-    // The StackView will trigger a visibility change when
-    // we're pushed onto it, causing our onVisibleChanged
-    // routine to run, but only if we start as invisible
-    visible: false
-
     SdlGamepadKeyNavigation {
         id: gamepadKeyNav
     }
 
-    onVisibleChanged: {
-        if (visible) {
+    StackView.onActivated: {
         gamepadKeyNav.setSettingsMode(true)
         gamepadKeyNav.enable()
     }
-        else {
-            gamepadKeyNav.disable()
-        }
-    }
 
-    Component.onDestruction: {
+    StackView.onDeactivating: {
+        gamepadKeyNav.disable()
         prefs.save()
     }
 

app/gui/StreamSegue.qml

@@ -15,11 +15,6 @@ Item {
 
     anchors.fill: parent
 
-    // The StackView will trigger a visibility change when
-    // we're pushed onto it, causing our onVisibleChanged
-    // routine to run, but only if we start as invisible
-    visible: false
-
     function stageStarting(stage)
     {
         // Update the spinner text
@@ -105,8 +100,7 @@ Item {
         id: gamepadKeyNav
     }
 
-    onVisibleChanged: {
-        if (visible) {
+    StackView.onActivated: {
         // Hide the toolbar before we start loading
         toolBar.visible = false
 
@@ -130,11 +124,11 @@ Item {
         // Run the streaming session to completion
         session.exec(Screen.virtualX, Screen.virtualY)
     }
-        else if (!quitAfter) {
-            // Show the toolbar again when we become hidden
+
+    StackView.onDeactivating: {
+        // Show the toolbar again
         toolBar.visible = true
     }
-    }
 
     Row {
         anchors.centerIn: parent