summaryrefslogtreecommitdiffstats
path: root/mobile-widgets/qml/DiveDetailsView.qml
diff options
context:
space:
mode:
authorGravatar Dirk Hohndel <dirk@hohndel.org>2020-03-30 09:34:12 -0700
committerGravatar Dirk Hohndel <dirk@hohndel.org>2020-03-31 08:42:20 -0700
commitaeb17946166eb162ac3efb32a8bacbef2d482824 (patch)
treedf2ad98fc3c97717784ad11ea2783ed5d934c815 /mobile-widgets/qml/DiveDetailsView.qml
parentc406ad83ea648e5559cecc7f22baab93347706f5 (diff)
downloadsubsurface-aeb17946166eb162ac3efb32a8bacbef2d482824.tar.gz
mobile/dive-details: remove the redundant 'map it' button
You can already get to the map by either clicking on the location text or on the left action button. This third way to get there reduces the available space for the location text, and can cause positioning issues with very long location texts creating three or more lines of text, which then overwrites the dateRow below. Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'mobile-widgets/qml/DiveDetailsView.qml')
-rw-r--r--mobile-widgets/qml/DiveDetailsView.qml17
1 files changed, 3 insertions, 14 deletions
diff --git a/mobile-widgets/qml/DiveDetailsView.qml b/mobile-widgets/qml/DiveDetailsView.qml
index 630ac3dbe..d3a811050 100644
--- a/mobile-widgets/qml/DiveDetailsView.qml
+++ b/mobile-widgets/qml/DiveDetailsView.qml
@@ -41,7 +41,7 @@ Item {
anchors {
left: parent.left
top: parent.top
- right: gpsButton.left
+ right: parent.right
margins: Math.round(Kirigami.Units.gridUnit / 2)
}
MouseArea {
@@ -53,22 +53,11 @@ Item {
}
}
}
- TemplateButton {
- id: gpsButton
- anchors.right: parent.right
- anchors.verticalCenter: locationText.verticalCenter
- enabled: gps !== ""
- text: qsTr("Map it")
- onClicked: {
- showMap()
- mapPage.centerOnDiveSite(diveSite)
- }
- }
Row {
id: dateRow
anchors {
left: locationText.left
- top: gpsButton.bottom
+ top: locationText.bottom
topMargin: Kirigami.Units.smallSpacing
bottom: numberText.bottom
@@ -100,7 +89,7 @@ Item {
color: subsurfaceTheme.textColor
anchors {
right: parent.right
- top: gpsButton.bottom
+ top: locationText.bottom
topMargin: Kirigami.Units.smallSpacing
}
}