diff options
| author |  Dirk Hohndel <dirk@hohndel.org> | 2020-03-30 09:34:12 -0700 |
|---|---|---|
| committer | Dirk Hohndel <dirk@hohndel.org> | 2020-03-31 08:42:20 -0700 |
| commit | aeb17946166eb162ac3efb32a8bacbef2d482824 (patch) | |
| tree | df2ad98fc3c97717784ad11ea2783ed5d934c815 /mobile-widgets/qml | |
| parent | c406ad83ea648e5559cecc7f22baab93347706f5 (diff) | |
| download | subsurface-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')
| -rw-r--r-- | mobile-widgets/qml/DiveDetailsView.qml | 17 |
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 } } |