<table><tr><td style="">dhaumann 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/D9217" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>I was just testing this. With this patch, a non-existing ~/test:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">Dolphin goes to /home/dh/test, and a red box tells the folder does not exist. This is better than the protocol-error message, so <a href="https://bugs.kde.org/show_bug.cgi?id=372813" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=372813</a> should be fixed.</li>
<li class="remarkup-list-item">In the file open-dialog: This expands also to /home/dh/test, but does not raise an error. So the contents of the listview remains the same.</li>
</ol>

<p>In summary:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">is certainly better with this patch.</li>
<li class="remarkup-list-item">file open dialog was possibly better without this patch, since "~/home" <enter> resulted in "/" and not silently in some location that does not exist. This should probably be improved in the file-open dialog itself?</li>
</ol>

<p>All in all, this patch is an improvement.</p></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D9217" rel="noreferrer">https://phabricator.kde.org/D9217</a></div></div><br /><div><strong>To: </strong>dfaure, emateli, elvisangelaccio, gregormi<br /><strong>Cc: </strong>dhaumann, broulik, Frameworks<br /></div>