<table><tr><td style="">broulik created this revision.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.
</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/D4838" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Since it represents a file, right clicking it should provide similar options found in a file manager.<br />
This allows to open the application with an application other than the default, e.g. if you want to edit the screenshot right away. It also offers service actions, such as send as email.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Took a screenshot, right clicked it, "Open Containing Folder" opened the parent folder of the screenshot.<br />
Open With also works.</p>
<p>I didn't add the plugin actions, which are "Compress" and "Link to activity" here which don't work in this context but I think KDE Connect is also a plugin action, so we probably should still add them? It would only be a <tt style="background: #ebebeb; font-size: 13px;">addPluginActionsTo(menu)</tt> call.</p>
<p>The notification times out independently of the menu because we technically leave the popup with our cursor and it would be quite complex to fix that. I did verify that when the popup closes the entries in the context menu still work.</p>
<p><a href="https://phabricator.kde.org/F2664593" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;" rel="noreferrer">F2664593: Screenshot_20170228_114914.png</a></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/D4838" rel="noreferrer">https://phabricator.kde.org/D4838</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>applets/notifications/package/contents/ui/ThumbnailStrip.qml<br />
applets/notifications/plugin/CMakeLists.txt<br />
applets/notifications/plugin/thumbnailer.cpp<br />
applets/notifications/plugin/thumbnailer.h</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>broulik, Plasma, VDG<br /><strong>Cc: </strong>plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>