[Differential] [Request, 4 lines] D3819: [Applicationjobs Data Engine] Expose job "destUrl"
broulik (Kai Uwe Broulik)
noreply at phabricator.kde.org
Mon Dec 26 22:00:44 UTC 2016
broulik created this revision.
broulik added a reviewer: Plasma.
broulik set the repository for this revision to R120 Plasma Workspace.
broulik added a dependency: D3818: [KUIserver] Forward "destUrl" all clients.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
This will eventually allow the notification applet to show a better finished notification.
Currently, it just uses whatever was set as second info label last. While this works for when copying a single file, it's utterly wrong when copying folders as this will be a random file that was copied last.
TEST PLAN
I can now (still wip local patches I need to clean up a bit)
- show the destination of a job while it's still figuring out what do copy (e.g. when no info label 2 is set, I'll show dest Url there instead)
- show Copying finished: /foo/bar instead of Copying finished: /foo/bar/some/random/file/that/was/copied/also.txt (need to figure out so a single file copied/received still shows the full path for easy access)
Depends on https://phabricator.kde.org/D3818
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D3819
AFFECTED FILES
dataengines/applicationjobs/kuiserverengine.cpp
dataengines/applicationjobs/kuiserverengine.h
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: broulik, #plasma
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161226/4a6b5382/attachment-0001.html>
More information about the Plasma-devel
mailing list