By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.

Bug 3108

Summary: KDCOP is missing application icon
Product: TDE Reporter: Jan Stolarek <jwstolarek>
Component: tdeadminAssignee: Michele Calgaro <michele.calgaro>
Status: RESOLVED FIXED    
Severity: trivial CC: bugwatch, jwstolarek, michele.calgaro, slavek.banko
Priority: P5    
Version: R14.0.x [Trinity]   
Hardware: Other   
OS: Linux   
Compiler Version: TDE Version String:
Application Version: Application Name:
Bug Depends on:    
Bug Blocks: 3085    
Attachments: Missing application icon

Description Jan Stolarek 2020-05-14 04:31:46 CDT
Created attachment 2960 [details]
Missing application icon

What the title says - instead of an application icon there's a generic question mark icon. See attached screenshot.
Comment 1 Michele Calgaro 2020-05-21 09:32:28 CDT
Confirmed. I am planning to fix it soon :-)
Comment 2 Michele Calgaro 2020-05-23 00:57:23 CDT
I am thinking of using this icon:
/opt/trinity/share/icons/crystalsvg/32x32/categories/applications-utilities.png
or
/opt/trinity/share/icons/crystalsvg/32x32/categories/preferences-system.png

Any preference? Or any other suggestion?
Comment 3 Jan Stolarek 2020-05-23 09:38:15 CDT
Personally I'd go with the second one.
Comment 4 Slávek Banko 2020-05-23 12:36:54 CDT
My suggestion is simple - we have already chosen the icon once - see the desktop file:

https://mirror.git.trinitydesktop.org/gitea/TDE/tdebase/src/branch/master/kdcop/kdcop.desktop#L4
Comment 5 Michele Calgaro 2020-05-23 23:37:32 CDT
> Confirmed. I am planning to fix it soon :-)

Ups, looks like I had misuunderstood this bug report. I use a custom icon for KDCOP since long ago, so I thought there was no icon at all. My bad.
Desktop icon was added by Greg in commit 6e32e1ee97b about 2 years ago. Going to use the same for the window then :-)
Comment 6 Michele Calgaro 2020-05-25 03:09:50 CDT
Fixed in commit 0844ba9d (R14.1) and a9e58754 (R14.0).
Thanks for reporting this.