D9112: Explicitly emit overlaysChanged in the setter rather than connecting to it
Kai Uwe Broulik
noreply at phabricator.kde.org
Sat Dec 2 17:10:03 UTC 2017
broulik created this revision.
broulik added a reviewer: Plasma.
Restricted Application added projects: Plasma, Frameworks.
Restricted Application added subscribers: Frameworks, plasma-devel.
REVISION SUMMARY
There's literally just one place where this thing changes so just emit the signal there and call it a day.
TEST PLAN
Overlays still change fine and show up on startup (the warning icon on non-exec desktop files)
REPOSITORY
R242 Plasma Framework (Library)
REVISION DETAIL
https://phabricator.kde.org/D9112
AFFECTED FILES
src/declarativeimports/core/iconitem.cpp
To: broulik, #plasma
Cc: plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171202/1d1155d5/attachment.html>
More information about the Plasma-devel
mailing list