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