Commit 6e846fdd authored by Nate Graham's avatar Nate Graham
Browse files

applets/taskmanager: always show album art for single-window tooltips

The Task Manager tooltips now have logic to avoid showing album art for
tooltips where the window title doesn't include the media name. This
makes sense for the case of a multi-window app, where it almost
perfectly solves the problem of the album art appearing on all of the
tooltips rather than only the relevant one.

However this check does not make sense for single-window apps, where
the only thing it can do is cause an app to erroneously hide the album
art for playing media because the app doesn't include the media name
in its window title. Elisa is one such app, and there are probably
others too.
parent 8af8aee5
Pipeline #102533 passed with stage
in 4 minutes and 54 seconds
......@@ -247,8 +247,9 @@ ColumnLayout {
Image {
id: albumArtImage
// also Image.Loading to prevent loading thumbnails just because the album art takes a split second to load
// don't show album art if window title doesn't include media title (eg we're in a different browser tab)
readonly property bool available: (status === Image.Ready || status === Image.Loading) && generateTitle().includes(track)
// don't show album art if window title doesn't include media title (eg we're in a different browser tab),
// but only if this is a tooltip for a multi-window app because otherwise there is no ambiguity
readonly property bool available: (status === Image.Ready || status === Image.Loading) && (!isGroup || generateTitle().includes(track))
anchors.fill: hoverHandler
// Indent by one pixel to make sure we never cover up the entire highlight
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment