[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