From 84d45485fd9c523142f6ff2ac5c18c3626adf15d Mon Sep 17 00:00:00 2001 From: Dirk Hohndel Date: Wed, 11 Mar 2020 16:54:46 -0700 Subject: mobile UI: remove duplicate StartPage This was a merge error in the dive list rewrite and brought in a code block that had moved, which caused issues with correctly switching to the dive list. Calling setupActions() moves to the diveListView (where it really belongs). Instead of messing with the visibility of components of the same page, we now simply switch out the shown page. Fixes #2667 Signed-off-by: Dirk Hohndel --- mobile-widgets/qml/main.qml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'mobile-widgets/qml/main.qml') diff --git a/mobile-widgets/qml/main.qml b/mobile-widgets/qml/main.qml index 2c9cd5b68..033bf6014 100644 --- a/mobile-widgets/qml/main.qml +++ b/mobile-widgets/qml/main.qml @@ -715,10 +715,10 @@ if you have network connectivity and want to sync your data to cloud storage."), onVisibleChanged: { if (visible) { pageStack.clear() - diveList.visible = false + } else { + showDiveList() } } - Component.onCompleted: { if (!visible) { manager.appendTextToLog("StartPage completed - showing the dive list") -- cgit v1.2.3-70-g09d2