Bug 467589

Summary: Low resolution icon in the notifications history
Product: [Plasma] plasmashell Reporter: Patrick Silva <bugseforuns>
Component: NotificationsAssignee: Plasma Bugs List <plasma-bugs-null>
Status: RESOLVED DOWNSTREAM    
Severity: normal CC: kde, nate
Priority: NOR    
Version First Reported In: 5.27.3   
Target Milestone: 1.0   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: screenshot
notification as a pop-up

Description Patrick Silva 2023-03-20 00:10:05 UTC
Created attachment 157431 [details]
screenshot

STEPS TO REPRODUCE
1. download something with Deluge torrent client
2. when the "finished torrent" appears, wait until it disappears
3. click on the notification applet in the system tray

OBSERVED RESULT
low resolution Deluge icon in the notifications history despite Deluge icon is crisp in the Kickoff menu.
Please see the attached screenshot.

EXPECTED RESULT
crisp icon in the notifications history

SOFTWARE/OS VERSIONS
Operating System: Arch Linux 
KDE Plasma Version: 5.27.3
KDE Frameworks Version: 5.104.0
Qt Version: 5.15.8
Graphics Platform: Wayland
Comment 1 Patrick Silva 2023-03-20 00:11:30 UTC
Correct step 2:
2. when the "finished torrent" notification appears, wait until it disappears
Comment 2 Nate Graham 2023-04-04 21:24:14 UTC
This icon is actually something that the app sends. It can send an icon name or it can send a pixmap. If it sends an icon name, then that icon gets looked up in the current icon theme. If it sends a pixmap, then it's up to the app to send one that's high enough resolution to avoid looking blurry.

Are you saying the icon looked good in the notification as a pop-up, but then it became blurry once the notification appeared in the history view?

If so, that would be unexpected and could be caused by a bug in our code. If it was always blurry, then this is a bug in Deluge itself, for not specifying a high enough resolution pixmap.
Comment 3 Patrick Silva 2023-04-04 21:42:52 UTC
Created attachment 157852 [details]
notification as a pop-up

The icon looks blurry in the notification as a pop-up too.
Comment 4 Nate Graham 2023-04-04 21:51:05 UTC
Ok thanks. So ultimately this is a Deluge bug, and if you want to follow up with the Deluge devs, you can ask them to either send an icon name instead of a pixmap, or else provide a higher-resolution pixmap.