From 74a5511b391ebf8120ba84c37b27a3a0cc6934a3 Mon Sep 17 00:00:00 2001 From: Dirk Hohndel Date: Sat, 15 Apr 2017 13:49:16 -0700 Subject: 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 --- mobile-widgets/qml/DiveDetails.qml | 2 -- 1 file changed, 2 deletions(-) (limited to 'mobile-widgets/qml/DiveDetails.qml') 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 } } -- cgit v1.2.3-70-g09d2