Bug 374707 - Low-res icons used for launcher tooltips
Summary: Low-res icons used for launcher tooltips
Status: RESOLVED FIXED
Alias: None
Product: libplasma
Classification: Frameworks and Libraries
Component: components (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR minor
Target Milestone: ---
Assignee: Marco Martin
URL:
Keywords:
: 365326 (view as bug list)
Depends on:
Blocks:
 
Reported: 2017-01-07 19:06 UTC by Hrvoje Senjan
Modified: 2017-01-11 22:15 UTC (History)
5 users (show)

See Also:
Latest Commit: https://commits.kde.org/plasma-framework/0a7ae2ce97c41ef86e3feb8086eadef01fb07078
Version Fixed In: 5.31.0
Sentry Crash Report:

Attachments
proof (118.35 KB, image/png)
2017-01-07 19:06 UTC, Hrvoje Senjan 		Details
proof #2 (105.38 KB, image/png)
2017-01-07 19:06 UTC, Hrvoje Senjan 		Details
proof #2 (121.97 KB, image/png)
2017-01-07 19:08 UTC, Hrvoje Senjan 		Details
appletsrc (29.98 KB, text/plain)
2017-01-09 09:05 UTC, Hrvoje Senjan 		Details
Show Obsolete (1) View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Hrvoje Senjan 2017-01-07 19:06:06 UTC 
Created attachment 103269 [details]
proof

See the screenshots
Comment 1 Hrvoje Senjan 2017-01-07 19:06:38 UTC 
Created attachment 103270 [details]
proof #2
Comment 2 Hrvoje Senjan 2017-01-07 19:08:37 UTC 
Created attachment 103271 [details]
proof #2
Comment 3 Eike Hein 2017-01-09 07:03:29 UTC 
Can you please attach your appletsrc?
Comment 4 Hrvoje Senjan 2017-01-09 09:05:40 UTC 
Created attachment 103300 [details]
appletsrc
Comment 5 Eike Hein 2017-01-09 09:26:10 UTC 
Thanks. Your appletsrc indicates discovery of the .desktop file works fine, so access to the hi-res asset should exist. This has to be a regression from the IconItem changes or the tooltip redesign. Kai, Roman?
Comment 6 Hrvoje Senjan 2017-01-09 10:13:48 UTC 
Bug is in plasma-framework, reverting to 0611e01 restores the proper icon rendering
Comment 7 Roman Gilg 2017-01-09 11:13:18 UTC 
You're right Hrvoje. Please test this patch: https://phabricator.kde.org/D4040
Comment 8 David Rosca 2017-01-09 11:38:19 UTC 
*** Bug 365326 has been marked as a duplicate of this bug. ***
Comment 9 Marco Martin 2017-01-11 17:27:37 UTC 
isthe problem still present after D4040 has been pushed?
Comment 10 Hrvoje Senjan 2017-01-11 22:15:47 UTC 
nope =)