Also handle opening the settings dialog through pushPage()

src/qml/Mobile/BottomToolbar.qml

@@ -49,10 +49,7 @@ Kirigami.NavigationTabBar {
             text: i18n("Settings")
             checked: "SettingsPage" === kastsMainWindow.currentPage
             onTriggered: {
-                applicationWindow().pageStack.clear()
+                pushPage("SettingsPage")
-                applicationWindow().pageStack.push("qrc:/SettingsPage.qml", {}, {
-                    title: i18n("Settings")
-                })
             }
         }
     ]

src/qml/main.qml

@@ -62,11 +62,18 @@ Kirigami.ApplicationWindow {
         }
     }
     function pushPage(page) {
+        if (page === "SettingsPage") {
+            pageStack.layers.clear()
+            pageStack.pushDialogLayer("qrc:/SettingsPage.qml", {}, {
+                title: i18n("Settings")
+            })
+        } else {
             pageStack.clear();
             pageStack.layers.clear();
             pageStack.push(getPage(page));
             currentPage = page;
         }
+    }
 
     Settings {
         id: settings
@@ -249,10 +256,7 @@ Kirigami.ApplicationWindow {
                     checked: currentPage == "SettingsPage"
                     onClicked: {
                         checked = false;
-                        kastsMainWindow.pageStack.layers.clear()
+                        pushPage("SettingsPage")
-                        kastsMainWindow.pageStack.pushDialogLayer("qrc:/SettingsPage.qml", {}, {
-                            title: i18n("Settings")
-                        })
                     }
                 }
             }