mobile/dive-details: restrict width of tags field

Having a lot of tags (or more precisely, a tags string that is very long) could
cause the width of the dive details view to extend past the width of the the
page. The txtTags label was missing a maximum width, and to make the result
more useful, I also added correct wrapping and elide to the mix (stupidly, we
had the wrap and width for the fixed name of the field ('Tags'), but not for
the user determined content of that field).

Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
This commit is contained in:
Dirk Hohndel 2020-06-08 11:03:10 -07:00
parent e00e72d430
commit 8a4a9382d3
2 changed files with 6 additions and 2 deletions

View file

@ -1,3 +1,4 @@
Mobile: correct UI issue with tags that were longer than page width
Mobile: add advanved option to import dives from local cloud cache directories
Mobile: fix broken editing of location, suit, buddy, and dive master
Mobile: fix missing translations on Android

View file

@ -519,9 +519,8 @@ Item {
TemplateLabelSmall {
text: qsTr("Tags:")
opacity: 0.6
wrapMode: Text.WrapAtWordBoundaryOrAnywhere
Layout.columnSpan: 3
Layout.maximumWidth: detailsView.col2Width + detailsView.col3Width
Layout.maximumWidth: detailsView.gridWidth
Layout.bottomMargin: 0
color: subsurfaceTheme.textColor
}
@ -532,6 +531,10 @@ Item {
id: txtTags
text: tags
wrapMode: Text.WrapAtWordBoundaryOrAnywhere
elide: Text.ElideRight
maximumLineCount: 3
Layout.maximumWidth: detailsView.gridWidth
height: Kirigami.Units.gridUnit * 3
Layout.columnSpan: 3
color: subsurfaceTheme.textColor
}