D8524: Add a signal notifying a change in the names of running activities.

[Ivan Čukić]

ivan added a comment.


  The activities cache is a private class meant to communicate directly to KAMD. The signals it contains are in the form `(activity id, something)` and are meant to be private. The `KActivities::Info` class was meant to wrap these signals into a nice OO API where an activity object (Info) sends signals that a particular property has changed.
  
  I don't like the idea of exposing the same thing multiple times with different APIs. In the 4.x era, we had one function with the `(id, something)` format and it was removed by request (during API review) during KF5-ization of KActivities library.
  
  One possible solution to all this (though I'm not sure I like it) is to add a class to KActivities API that keeps maintains a collection of Info objects (exposing the cache in a sense) - it could have `activityAdded(Info*)`, `activityRemoved(Info*)` and `activityChanged(Info*)` or something similar.

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D8524

To: hein, #plasma, davidedmundson
Cc: davidedmundson, anthonyfieroni, ivan, plasma-devel, 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/20171117/6bf07317/attachment.html>