<table><tr><td style="">broulik added a comment.
</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/D9255" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Or should we remove the "Öffnen/Open" button?</p></blockquote>

<p>Good idea. However, the "Open" button is created by the application itself and there's no way of knowing to the application whether the notification server supports that thumbnail strip that is proprietary to Plasma and non-standard. While FDO specifies a "Capabilities" property that a client can query to see what it supports, that isn't accessible through <tt style="background: #ebebeb; font-size: 13px;">KNotifications</tt>. In short: if we remove the "Open" button on other desktops the user won't get any way of accessing the screenshot.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D9255#inline-41781" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">romangg</span> wrote in <span style="color: #4b4d51; font-weight: bold;">ThumbnailStrip.qml:113</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Since now the thumbnail is clickable, remove.</p>

<p style="padding: 0; margin: 8px;">Also could you give the thumbnail an effect to show that it is clickable? Best would be imo a frame in highlight color, which switches to the context menu button, when mouse cursor moves from thumbnail to inside of context menu button area. This would be consistent with other elements of the workspace (with breeze).</p>

<p style="padding: 0; margin: 8px;">But since we have not yet decided on a consistent highlighting scheme in general, you can also try other styles.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">The thumbnail has *always* been clickable.</p>

<p style="padding: 0; margin: 8px;">The open hand cursor is merely an indicator to show that it's draggable, the very reason this feature exists.</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D9255#inline-41785" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">romangg</span> wrote in <span style="color: #4b4d51; font-weight: bold;">thumbnailer.cpp:189</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">The following code positions the menu according to the button. But it still covers half of it. The context menu should not cover the button at all. Can you somehow use PlasmaComponents.Menu? It already has suitable placement code covering special cases (not enough space etc.).</p>

<p style="padding: 0; margin: 8px;">Otherwise copy the placement code from its C++ part and use it here.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">The problem is not the placing but the fact that the actions are populated asynchronously, you can observe the same in Dolphin where the context menu sometimes (very rarely, actually) shows up detached from the cursor position.</p></div></div></div></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/D9255" rel="noreferrer">https://phabricator.kde.org/D9255</a></div></div><br /><div><strong>To: </strong>broulik, Plasma, VDG, ngraham, romangg<br /><strong>Cc: </strong>januz, romangg, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>