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:
Version Fixed In: 5.31.0


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

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 =)