<table><tr><td style="">rkflx edited reviewers, added: elvisangelaccio; removed: rkflx.<br />rkflx added a subscriber: elvisangelaccio.<br />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/D12206">View Revision</a></tr></table><br /><div><div><p><a href="https://phabricator.kde.org/p/ngraham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ngraham</a> The way I understood <a href="https://phabricator.kde.org/p/elvisangelaccio/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@elvisangelaccio</a> was that the possibility to open a folder with another application should stay, but featured less prominently (i.e. like before). My ping was mainly to get the conversation going again, because it looked like Elvis' concern was being ignored.</p>

<p>I would not agree with completely removing that functionality, because it <em>is</em> actually useful sometimes, e.g. for KDirStat/Filelight etc., and has been a feature even in Konqueror since more than a decade.</p>

<p>+1 for only showing the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Open With</span></span></span> menu, without an additional top app. (But make sure to consolidate the number of separators…)</p>

<p><a href="https://phabricator.kde.org/p/elvisangelaccio/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@elvisangelaccio</a> Please comment, so we'll know what you had in mind ;)</p>

<hr class="remarkup-hr" />

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>According to a comment in the code, <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Open With</span></span></span> was never supposed to show up for folders in the first place, but a bug prevented this from working: the value of <tt style="background: #ebebeb; font-size: 13px;">firstItem.url().isLocalFile()</tt> was also checked, but this always returns true for local directories, not just local files.</p></blockquote>

<p>Are you sure? The API docs say that "A URL is a local file path if the scheme is "file".", i.e. it also applies to directories. The way I interpret <tt style="background: #ebebeb; font-size: 13px;">not very useful, especially for remote folders</tt>, the way the condition is written and how it works in reality, is that only for remote folders the entry should be hidden.</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/D12206">https://phabricator.kde.org/D12206</a></div></div><br /><div><strong>To: </strong>ngraham, Frameworks, elvisangelaccio<br /><strong>Cc: </strong>elvisangelaccio, Zren, michaelh, ngraham, bruns<br /></div>