<table><tr><td style="">rkflx 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/D11733">View Revision</a></tr></table><br /><div><div><p>TBH I'm not convinced that's something which should be added. That's a pretty weird workflow in general, and the bug reporter's use case of selecting a file in a web browser for upload essentially wants the existing <kbd style="display: inline-block; min-width: 1em; padding: 4px 5px 5px; font-weight: normal; font-size: 0.8rem; text-align: center; text-decoration: none; line-height: 0.6rem; border-radius: 3px; box-shadow: inset 0 -1px 0 rgba(71, 87, 120, 0.08); user-select: none; background: #f7f7f7; border: 1px solid #C7CCD9;">F11</kbd> preview feature and perhaps a better preview feature in the web application itself. Also note that you could just drag-and-drop the file onto Dolphin, which should then open the default app, or onto Gwenview's launcher etc.</p>

<p>My concern is that the naming as well as the concept itself clashes with the regular <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Open</span></span></span> button while doing something different. Also, this recursive opening thing has all sorts of issues. Instead of cluttering the context menu with more entries, going the extra step via <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Open Containing Folder</span></span></span> is good enough IMO.</p>

<p>Maybe I'll change my mind if you can come up with a very compelling use case which is so great it's worth living with the disadvantages. For now my (i.e. take this with a grain of salt) recommendation would be to not add this.</p>

<hr class="remarkup-hr" />

<p>Instead of adding <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Open</span></span></span> buttons everywhere, a more elegant and generic approach would perhaps be to tackle this from a Plasma angle: Currently we have KRunner for typing things and Klipper's actions for (automatically) acting on content added to the clipboard. I could imagine a dynamically opening Plasmoid tailored for drag-and-drop operations could complement this nicely, e.g. upon initiating a drag operation it could present various drop targets like opening in apps, deletion, quick-preview, adding to the stash, tagging, user-configurable scripts, access to Dolphin's mimetype-specific actions etc.</p>

<p>Another (less exciting) idea would be to add what this patch does to the preview itself, which currently has no buttons and no context menu.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11733">https://phabricator.kde.org/D11733</a></div></div><br /><div><strong>To: </strong>ngraham, Dolphin, Frameworks<br /><strong>Cc: </strong>rkflx, broulik, michaelh, ngraham<br /></div>