wallpapers/slideshowpackage: fix tooltip so it actually appears

We had disabled hover on the list item, so it would never become
visible. In addition, its visibility condition was wrong as we only want
it to become visible for list items that have elided text.

BUG: 459052
FIXED-IN: 5.26
wilder-5.26
Nate Graham 4 years ago
parent c67f90a358
commit fe9f664963
  1. 9
      wallpapers/image/slideshowpackage/contents/ui/SlideshowComponent.qml

@ -191,8 +191,11 @@ ColumnLayout {
hoverEnabled: false hoverEnabled: false
contentItem: Kirigami.BasicListItem { contentItem: Kirigami.BasicListItem {
width: slidePathsView.width - baseListItem.overlayWidth width: slidePathsView.width - baseListItem.overlayWidth
// Don't need a highlight or hover effects // Don't want a background highlight effect, but we can't just
hoverEnabled: false // set hoverEnabled to false, since then the tooltip will
// never appear!
activeBackgroundColor: "transparent"
activeTextColor: Kirigami.Theme.textColor
separatorVisible: false separatorVisible: false
// Header: the folder // Header: the folder
@ -207,7 +210,7 @@ ColumnLayout {
} }
QQC2.ToolTip.text: modelData QQC2.ToolTip.text: modelData
QQC2.ToolTip.visible: hovered QQC2.ToolTip.visible: hovered && (labelItem.truncated || subtitleItem.truncated)
QQC2.ToolTip.delay: Kirigami.Units.toolTipDelay QQC2.ToolTip.delay: Kirigami.Units.toolTipDelay
} }

Loading…
Cancel
Save