diff options
| author |  Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2019-08-30 12:38:25 +0200 |
|---|---|---|
| committer |  Dirk Hohndel <dirk@hohndel.org> | 2019-09-06 11:48:47 -0700 |
| commit | 28cb75b73d3d0fa4cd8dcf3aa5884e93cb13a5d2 (patch) | |
| tree | 7ddc15816df96e7eb16f8ac6460ff6d298a6023b /map-widget/qml | |
| parent | f818ac3352b5a66d6fcc452ddc5ebb84a18bb29a (diff) | |
| download | subsurface-28cb75b73d3d0fa4cd8dcf3aa5884e93cb13a5d2.tar.gz | |
Map: explicitly reload selected map on click
When clicking on a flag
1) The QML would call MapLocationModel::setSelected() with
fromClick = true
2) MapLocationModel::setSelected() would emit a signal
selectedLocationChanged()
3) MapWidgetHelper would catch that signal and do the actual
processing.
Other functions would call MapLocationModel::setSelected() with
fromClick = false, which would not emit the selectedLocationChanged()
signal.
Detangle this a bit by calling the selectedLocationChanged() function
directly from QML and remove the fromClick parameter.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'map-widget/qml')
| -rw-r--r-- | map-widget/qml/MapWidget.qml | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/map-widget/qml/MapWidget.qml b/map-widget/qml/MapWidget.qml index 602b7c338..42fc0ab88 100644 --- a/map-widget/qml/MapWidget.qml +++ b/map-widget/qml/MapWidget.qml @@ -69,8 +69,10 @@ Item { drag.target: (mapHelper.editMode && mapHelper.model.isSelected(model.divesite)) ? mapItem : undefined anchors.fill: parent onClicked: { - if (!mapHelper.editMode && model.divesite) - mapHelper.model.setSelected(model.divesite, true) + if (!mapHelper.editMode && model.divesite) { + mapHelper.model.setSelected(model.divesite) + mapHelper.selectedLocationChanged(model.divesite) + } } onDoubleClicked: map.doubleClickHandler(mapItem.coordinate) onReleased: { |