D21061: [Task Manager] Use libnotificationmanager for notification badges and application job reporting

Kai Uwe Broulik noreply at phabricator.kde.org
Tue May 7 08:57:27 BST 2019


broulik created this revision.
broulik added reviewers: Plasma, hein, mvourlakos.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.

REVISION SUMMARY
  This removes the setting from the individual plasmoid and has it follow global settings in the new KCM.
  Since the new `JobsModel` also emits application jobs via the Unity launcher API, the codepath for dealing with jobs is dropped and instead it only listens to Unity signals now.

TEST PLAN
  Depends on D20265 <https://phabricator.kde.org/D20265>
  There's still something funky going on with shared pointer ownership, when I disable notification applet jobs the model is deleted for some reason... also, do not disturb mode doesnt auto expire as I dont check the time periodically (might slap in the time dataengine like I do in notifications). It cannot tell between genuine badges and job badges, so when you disable job tracking you stil get a badge of "1" thing going on. Also, since the setting is global now, the smart launcher stuff is always loaded but it's just one more qobject per task and the QML bits for progress and badges are behind loaders already, only loading when there is a progress or badge to be displayed.
  
  Other than that, works well, even with Latte Dock.
  
  - Copied files in dolphin, progress showed up
  - Set a badge on an app, showed up
  - Changed settings, they updated the thing live

REPOSITORY
  R119 Plasma Desktop

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

AFFECTED FILES
  CMakeLists.txt
  applets/taskmanager/CMakeLists.txt
  applets/taskmanager/package/contents/config/main.xml
  applets/taskmanager/package/contents/ui/ConfigAppearance.qml
  applets/taskmanager/package/contents/ui/Task.qml
  applets/taskmanager/plugin/smartlaunchers/smartlauncherbackend.cpp
  applets/taskmanager/plugin/smartlaunchers/smartlauncherbackend.h
  applets/taskmanager/plugin/smartlaunchers/smartlauncheritem.cpp
  applets/taskmanager/plugin/smartlaunchers/smartlauncheritem.h

To: broulik, #plasma, hein, mvourlakos
Cc: plasma-devel, jraleigh, GB_2, 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/20190507/a6695189/attachment.html>


More information about the Plasma-devel mailing list