<table><tr><td style="">broulik created this revision.<br />broulik added a reviewer: Plasma.<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/D23293">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This implements a new <tt style="background: #ebebeb; font-size: 13px;">JobViewV2</tt> which uses extensible variant maps rather than individual function calls, allowing for compression of calls and extensibility.<br />
The new API uses <tt style="background: #ebebeb; font-size: 13px;">infoMessage</tt> correctly as a state message, e.g. "Connecting to host" rather than mixing it with the "Copying..." heading.<br />
It also supports an "immediate" flag that the caller can use to indicate progress should be immediately shown, so in cases where the user is likely to want to use the file afterwards (e.g. download through p-b-i or receiving a file through KDE Connect) a job popup is shown even for small/short transfers.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>This has to go into 5.16 branch as I implemented the interface in 5.16 but it is defunct, so when we ever start using it in KIO it will see the interface but it will not work.</p>

<p>Used a WIP branch of kjobwidgets/kio and been using the new API without a hitch.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D23293">https://phabricator.kde.org/D23293</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>libnotificationmanager/job.cpp<br />
libnotificationmanager/job_p.cpp<br />
libnotificationmanager/job_p.h<br />
libnotificationmanager/jobsmodel_p.cpp</div></div></div><br /><div><strong>To: </strong>broulik, Plasma<br /><strong>Cc: </strong>plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>