<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: Plasma, VDG.<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/D25782">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This adds a file icon to a finished job for a single total file which can be dragged anywhere for convenience.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Often I download a file and then want to send it to someone, so being able to drag it off the notification similar to how you can do it with the screenshot notification is super useful.</p>

<p>The button glows on hover and has the open hand cursor to indicate it can be dragged<br />
<a href="https://phabricator.kde.org/F7802057" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7802057: Screenshot_20191206_125415.png</a><br />
Clicking it opens the file and right click opens a file context menu. It is kinda redundant with the "Open" and menu button but I think something that looks like a file should be have like a file and those visual actions should be kept for accessibility.</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/D25782">https://phabricator.kde.org/D25782</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>applets/notifications/notificationapplet.cpp<br />
applets/notifications/notificationapplet.h<br />
applets/notifications/package/contents/ui/JobItem.qml<br />
applets/notifications/package/contents/ui/NotificationItem.qml<br />
applets/notifications/package/contents/ui/main.qml</div></div></div><br /><div><strong>To: </strong>broulik, Plasma, VDG<br /><strong>Cc: </strong>plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>