status-im
/
status-qml
mirror of
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

fix(StatusListItem): don't set width on title item 

There was an explicit `width` introduced on `statusListItemTitle`, most likely
to make room for it when `titleAsideText` is supplied.

This unfortunately causes the title get a very small width, resulting in broken
UI.

Since we can assume that there should be enough space for the titleAsideText
when it's used, we probably don't have to calculate a fixed width for the title
in the first place.

This commit therefore removes that explicit setting.
This commit is contained in:
Pascal Precht 2021-08-26 12:18:56 +02:00 committed by Pascal Precht
parent a4178bd6dc
commit ee5ec7b3db
2 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
sandbox/ListItems.qml
View File

@ -289,6 +289,11 @@ CExPynn1gWf9bx498P7/nzPcxEzGExhBdJGYihtAYQlO+tUZvqrPbqeudo5iJGEJjCE15a3VtodH3q2I
icon.background.color: "orange" icon.background.color: "orange"
} }
StatusListItem {
title: "Title"
titleAsideText: "test"
}
StatusListItem { StatusListItem {
title: "Title" title: "Title"
icon.name: "delete" icon.name: "delete"

2
src/StatusQ/Components/StatusListItem.qml
View File

@ -179,8 +179,6 @@ Rectangle {
StatusBaseText { StatusBaseText {
id: statusListItemTitle id: statusListItemTitle
text: statusListItem.title text: statusListItem.title
width: contentWidth < (parent.width - statusListItemTitleAsideText.contentWidth) ?
contentWidth : (parent.width - statusListItemTitleAsideText.contentWidth)
font.pixelSize: 15 font.pixelSize: 15
height: visible ? contentHeight : 0 height: visible ? contentHeight : 0
wrapMode: Text.WrapAtWordBoundaryOrAnywhere wrapMode: Text.WrapAtWordBoundaryOrAnywhere