D15599: Port the "Switch Desktop" containment action to libtaskmanager
Eike Hein
noreply at phabricator.kde.org
Wed Oct 17 19:11:19 BST 2018
hein updated this revision to Diff 43810.
hein added a comment.
- Fix another arg() call
- Do delete+remove in one step with take()
- Fix loop condition
Addressing other review comments:
- More change tracking is not needed because of the later loop that indiscriminately calls setText/setData on all actions
- The suggested qDeleteAll+resize method doesn't work because QHash has no resize()
REPOSITORY
R120 Plasma Workspace
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D15599?vs=43754&id=43810
BRANCH
arcpatch-D15599
REVISION DETAIL
https://phabricator.kde.org/D15599
AFFECTED FILES
containmentactions/switchdesktop/CMakeLists.txt
containmentactions/switchdesktop/desktop.cpp
containmentactions/switchdesktop/desktop.h
containmentactions/switchwindow/switch.cpp
To: hein, mart, davidedmundson
Cc: zzag, broulik, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20181017/f38df340/attachment-0001.html>
More information about the Plasma-devel
mailing list