Bug 429646 - Cover art no longer displayed in system tray
Summary: Cover art no longer displayed in system tray
Status: CONFIRMED
Alias: None
Product: amarok
Classification: Applications
Component: general (other bugs)
Version First Reported In: kf5
Platform: Other Linux
: NOR normal
Target Milestone: kf5
Assignee: Amarok Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-11-25 14:59 UTC by Mark Fraser
Modified: 2020-12-07 10:42 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments
System tray (52.89 KB, image/png)
2020-11-25 14:59 UTC, Mark Fraser
Details
Systray tooltip album cover art prototype (288.82 KB, image/png)
2020-12-07 10:42 UTC, Tuomas Nurmi
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Mark Fraser 2020-11-25 14:59:27 UTC
Created attachment 133646 [details]
System tray

SUMMARY
If I hover over the Amarok icon in the system tray, I used to see the track details and the cover art. Now I just see unformatted text.

STEPS TO REPRODUCE
1. Start Amarok playing
2. Move pointer to Amarok system tray icon
3. Observe result

OBSERVED RESULT


EXPECTED RESULT


SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 5.19.5
KDE Frameworks Version: 5.74.0
Qt Version: 5.14.2

ADDITIONAL INFORMATION
Comment 1 Myriam Schweingruber 2020-11-26 17:03:39 UTC
indeed, the formatting is gone, might be a side effect of the refactoring of the Context View. Current build is 17.9.2020
Comment 2 Tuomas Nurmi 2020-12-06 11:16:43 UTC
Formatting would be fixed by my pending merge request at https://invent.kde.org/multimedia/amarok/-/merge_requests/17 . The cover art is a bit more complicated, as it is correctly set as KStatusNotifierItem's ToolTipIcon, however, after reading the Plasma code a bit, it seems that at least Plasma systray does not use ToolTipIcon for anything at all, thus it is not visible.

The cover could easily be added to the tooltip, however I need to figure out how to format the tooltip nicely then.
Comment 3 Tuomas Nurmi 2020-12-07 10:42:13 UTC
Created attachment 133915 [details]
Systray tooltip album cover art prototype

Ok, according to #plasma, the tooltip icon (containing the cover art) is indeed ignored in Plasma systray. The cover could be added to the tooltip, however, since the tooltip uses QML format Text.StyledText, positioning options are quite limited. If it is on same line with a line of text, the line spacing gets stretched. Making it very big (similar to attached image, but perhaps even bigger, to cover the whole tooltip width) looks almost good, however causes the tooltip to be very tall. A bit tricky to make it look good with the available options.