From aeb17946166eb162ac3efb32a8bacbef2d482824 Mon Sep 17 00:00:00 2001 From: Dirk Hohndel Date: Mon, 30 Mar 2020 09:34:12 -0700 Subject: 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 --- mobile-widgets/qml/DiveDetailsView.qml | 17 +++-------------- 1 file changed, 3 insertions(+), 14 deletions(-) (limited to 'mobile-widgets/qml/DiveDetailsView.qml') 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 } } -- cgit v1.2.3-70-g09d2