Review Request 126016: fix: properly recognise Plasma 5 KCM modules (wmClass=kcmshell5)

[Johan Ouwerkerk]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126016/
-----------------------------------------------------------

(Updated Feb. 8, 2017, 4:02 p.m.)


Status
------

This change has been discarded.


Review request for Plasma.


Repository: plasma-workspace


Description
-------

Previously the taskmanager library contained a special case logic for windows of KDE-4 KCM modules (only).
These modules were recognised by finding wmClass=Kcmshell4.
This logic is extended to cover kcmshell5 windows as well, meaning that KCMs written for Plasma 5 are properly recognised now.
The net benefit is that these KCMs are displayed in the task manager with their proper KCM program icons.

This patch can be pulled from the kcmshell5-task-url-fixes branch at: git at github.com:cmacq2/plasma-workspace.git


Diffs
-----

  libtaskmanager/taskitem.cpp 3b2a4188fc8ed087a331999aee279ecd919c628e 

Diff: https://git.reviewboard.kde.org/r/126016/diff/


Testing
-------

Built with kdesrc-build, and tested using: `plasmawindowed org.kde.plasma.icontasks`.
I checked the change works as expected by running `which kcmshell5` style as well as `kcmshell5 style`: the icon of the window matches that in system settings (as expected).


Thanks,

Johan Ouwerkerk

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170208/a7c021d1/attachment.html>