<table><tr><td style="">ngraham created this revision.<br />ngraham added reviewers: Dolphin, Frameworks.<br />Restricted Application added a project: Frameworks.<br />ngraham 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/D11733">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This patch adds an <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> menu item to the open/save dialogs' context menu.</p>

<p>FEATURE: 392329</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>I am not 100% on this feature. The bug report requested the feature for previewing files, but maybe the issue is discoverability of the inline and aside previews? This patch is offered as much for discussion as to implement the feature and close the bug, but it does work:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item"><span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Right-click on a file</span></span><span style="color: #92969D;"> → </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> -> Opens the file in the default app for it</li>
<li class="remarkup-list-item"><span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Right-click on a folder</span></span><span style="color: #92969D;"> → </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> -> Opens the folder</li>
<li class="remarkup-list-item"><span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Right-click on the background</span></span><span style="color: #92969D;"> → </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> -> Opens the parent folder</li>
<li class="remarkup-list-item">One issue is that if the item opens in the very same app that invoked the open/save dialog, the resulting window can't actually be interacted with or closed because the dialog is modal.</li>
</ul>

<p>[image goes here]</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>BRANCH</strong><div><div>add-open-to-context-menu (branched from master)</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>AFFECTED FILES</strong><div><div>src/filewidgets/kdiroperator.cpp<br />
src/filewidgets/kdiroperator.h</div></div></div><br /><div><strong>To: </strong>ngraham, Dolphin, Frameworks<br /><strong>Cc: </strong>michaelh, ngraham<br /></div>