Bug 430054

Summary: Missing windows icons in windows runner
Product: [Plasma] kwin Reporter: Kai Uwe Broulik <kde>
Component: generalAssignee: KWin default assignee <kwin-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: nate
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: https://invent.kde.org/plasma/kwin/commit/5edfccac127d054f7dfd47461b501dabbdd8bcf5 Version Fixed/Implemented In: 5.21
Sentry Crash Report:

Description Kai Uwe Broulik 2020-12-05 20:20:07 UTC 
SUMMARY
There are no icons for windows in the windows runner anymore on X11.

STEPS TO REPRODUCE
1. Have a window with title "foo"
2. Search for "foo" in KRunner

OBSERVED RESULT
A window entry "foo" with no icon

EXPECTED RESULT
A window entry "foo" with the appropriate icon

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 5.20.80
KDE Frameworks Version: 5.77.0
Qt Version: 5.15.1

ADDITIONAL INFORMATION
There is no pixmap serialization happening in the runner whatsoever, just 

match.iconName = client->icon().name();

which requires themed window icons which we don't have on X.
Comment 1 Bug Janitor Service 2020-12-17 19:06:53 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/558
Comment 2 Kai Uwe Broulik 2020-12-29 21:22:53 UTC 
Git commit 5edfccac127d054f7dfd47461b501dabbdd8bcf5 by Kai Uwe Broulik.
Committed on 29/12/2020 at 21:22.
Pushed by broulik into branch 'master'.

KRunner integration: Send window pixmap along

If there is no icon name available which is very often
the case on X11.

M  +39   -1    plugins/krunner-integration/dbusutils_p.h
M  +16   -0    plugins/krunner-integration/windowsrunnerinterface.cpp

https://invent.kde.org/plasma/kwin/commit/5edfccac127d054f7dfd47461b501dabbdd8bcf5