Dialog pop should be done via accept destination

piggz · May 17, 2024 · 1ca6805 · 1ca6805
1 parent 8efe634
commit 1ca6805
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 12 deletions.
diff --git a/ui/qml/components/platform.kirigami/DialogPL.qml b/ui/qml/components/platform.kirigami/DialogPL.qml
@@ -35,12 +35,14 @@ PagePL {
 
     onAccepted: {
         if (acceptDestination) {
-            if (acceptDestinationPop)
+            if (acceptDestinationPop) {
                 app.pages.pop(acceptDestination);
-            else
-                app.push(acceptDestination);
-        } //else
-            //app.pages.pop();
+            } else {
+                app.pages.push(acceptDestination);
+            }
+        } else {
+            app.pages.pop();
+        }
     }
 
     function accept() {

diff --git a/ui/qml/pages/FirstPage.qml b/ui/qml/pages/FirstPage.qml
@@ -11,21 +11,18 @@ PagePL {
 
     function unpairAccepted() {
         DaemonInterfaceInstance.disconnect();
-        app.pages.push(Qt.resolvedUrl("./PairSelectDeviceType.qml"));
     }
 
     pageMenu: PageMenuPL {
         PageMenuItemPL {
             text: qsTr("Pair with watch")
             onClicked: {
-                var page = AmazfishConfig.pairedAddress
-                        ? "UnpairDeviceDialog.qml"
-                        : "PairSelectDeviceType.qml"
-
-                var obj = app.pages.push(Qt.resolvedUrl(page));
-
                 if (AmazfishConfig.pairedAddress) {
+                    var obj = app.pages.push(Qt.resolvedUrl("UnpairDeviceDialog.qml"));
+                    obj.acceptDestination = Qt.resolvedUrl("PairSelectDeviceType.qml")
                     obj.accepted.connect(unpairAccepted);
+                } else {
+                    app.pages.push(Qt.resolvedUrl("PairSelectDeviceType.qml"));
                 }
             }
         }