<table><tr><td style="">rjvbb 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/D16648">View Revision</a></tr></table><br /><div><div><p>A thought: shouldn't the <tt style="background: #ebebeb; font-size: 13px;">KRun::run*</tt> functions use <tt style="background: #ebebeb; font-size: 13px;">QCoreApplication::applicationFilePath()</tt> instead of invoking "dolphin" and hope the path leads to the same application?</p>

<p>I haven't read through the entire review to see if there has been discussion of this (beyond the allusions to having different versions in the test plan) nor what has been decided, but:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">I'd expect "open in new window" to behave as if it opened a new window in the same application (and not, for instance, give me the KDE4 version if for some reason I'm testing Dolphin 5 on a system that still runs a Plasma 4 desktop)</li>
<li class="remarkup-list-item">typically there will not be a dolphin executable on the path when running on a Mac, nor on MS Windows. On the latter "open in new window" seems to work (as long as the application runs in its installation directory?), on the former the feature fails with the expected error that dolphin cannot be found.</li>
</ul>

<p>(I'm bringing this up here because it's the latest discussed, relevant change to <tt style="background: #ebebeb; font-size: 13px;">Dolphin::openNewWindow()</tt>.)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R318 Dolphin</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16648">https://phabricator.kde.org/D16648</a></div></div><br /><div><strong>To: </strong>feverfew, Dolphin, elvisangelaccio, thomasp<br /><strong>Cc: </strong>rjvbb, thomasp, zzag, dfaure, fvogt, fikrim, magar, fbg13, davidedmundson, kwin, ngraham, elvisangelaccio, anthonyfieroni, kfm-devel, azyx, nikolaik, pberestov, iasensio, aprcela, fprice, fbampaloukas, alexde, Codezela, feverfew, meven, spoorun, navarromorales, firef, andrebarros, emmanuelp, rdieter, mikesomov<br /></div>