diff options
| author |  Dirk Hohndel <dirk@hohndel.org> | 2017-04-15 13:49:16 -0700 |
|---|---|---|
| committer | Dirk Hohndel <dirk@hohndel.org> | 2017-04-15 13:50:18 -0700 |
| commit | 74a5511b391ebf8120ba84c37b27a3a0cc6934a3 (patch) | |
| tree | 23f21a303f51f373fa6ce214d87f0a0bae272263 | |
| parent | cc6c41c8464a10c8d5a0d5a3cdf6c7d03b95ed57 (diff) | |
| download | subsurface-74a5511b391ebf8120ba84c37b27a3a0cc6934a3.tar.gz | |
QML UI: fix broken delete from dive details page
Trying to close the non-existing context drawer caused the function
to abort (and consequently not to delete the dive).
Fixes #309
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
| -rw-r--r-- | mobile-widgets/qml/DiveDetails.qml | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/mobile-widgets/qml/DiveDetails.qml b/mobile-widgets/qml/DiveDetails.qml index fc1fd9548..b854b4432 100644 --- a/mobile-widgets/qml/DiveDetails.qml +++ b/mobile-widgets/qml/DiveDetails.qml @@ -75,7 +75,6 @@ Kirigami.Page { text: qsTr("Delete dive") iconName: "trash-empty" onTriggered: { - contextDrawer.close() var deletedId = diveDetailsListView.currentItem.modelData.dive.id var deletedIndex = diveDetailsListView.currentIndex manager.deleteDive(deletedId) @@ -84,7 +83,6 @@ Kirigami.Page { function() { diveDetailsListView.currentIndex = manager.undoDelete(deletedId) ? deletedIndex : diveDetailsListView.currentIndex }); - contextDrawer.close() // at least one iPhone user has the drawer pop open after delete } } |