From 8a4a9382d39fc5074ca36d51b4730e77806e3405 Mon Sep 17 00:00:00 2001 From: Dirk Hohndel Date: Mon, 8 Jun 2020 11:03:10 -0700 Subject: 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 --- mobile-widgets/qml/DiveDetailsView.qml | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) (limited to 'mobile-widgets') diff --git a/mobile-widgets/qml/DiveDetailsView.qml b/mobile-widgets/qml/DiveDetailsView.qml index 54e188fe9..bd6fbbd89 100644 --- a/mobile-widgets/qml/DiveDetailsView.qml +++ b/mobile-widgets/qml/DiveDetailsView.qml @@ -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 } -- cgit v1.2.3-70-g09d2