<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: Plasma, hein, mvourlakos.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />broulik requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D21061">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This removes the setting from the individual plasmoid and has it follow global settings in the new KCM.<br />
Since the new <tt style="background: #ebebeb; font-size: 13px;">JobsModel</tt> 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.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Depends on <a href="https://phabricator.kde.org/D20265" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D20265</a><br />
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.</p>

<p>Other than that, works well, even with Latte Dock.</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Copied files in dolphin, progress showed up</li>
<li class="remarkup-list-item">Set a badge on an app, showed up</li>
<li class="remarkup-list-item">Changed settings, they updated the thing live</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D21061">https://phabricator.kde.org/D21061</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>CMakeLists.txt<br />
applets/taskmanager/CMakeLists.txt<br />
applets/taskmanager/package/contents/config/main.xml<br />
applets/taskmanager/package/contents/ui/ConfigAppearance.qml<br />
applets/taskmanager/package/contents/ui/Task.qml<br />
applets/taskmanager/plugin/smartlaunchers/smartlauncherbackend.cpp<br />
applets/taskmanager/plugin/smartlaunchers/smartlauncherbackend.h<br />
applets/taskmanager/plugin/smartlaunchers/smartlauncheritem.cpp<br />
applets/taskmanager/plugin/smartlaunchers/smartlauncheritem.h</div></div></div><br /><div><strong>To: </strong>broulik, Plasma, hein, mvourlakos<br /><strong>Cc: </strong>plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>