diff options
| author |  Dirk Hohndel <dirk@hohndel.org> | 2020-03-11 16:54:46 -0700 |
|---|---|---|
| committer | Dirk Hohndel <dirk@hohndel.org> | 2020-03-12 07:07:48 -0700 |
| commit | 84d45485fd9c523142f6ff2ac5c18c3626adf15d (patch) | |
| tree | 874db93e4168ffc0fc7208d73fd4cf16b2a49512 /mobile-widgets/qml/main.qml | |
| parent | 0d1510929f9b958c5b5d307bdea133fbf578fc16 (diff) | |
| download | subsurface-84d45485fd9c523142f6ff2ac5c18c3626adf15d.tar.gz | |
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 <dirk@hohndel.org>
Diffstat (limited to 'mobile-widgets/qml/main.qml')
| -rw-r--r-- | mobile-widgets/qml/main.qml | 4 |
1 files changed, 2 insertions, 2 deletions
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") |