<table><tr><td style="">sandsmark requested changes to this revision.<br />sandsmark added a comment.<br />This revision now requires changes to proceed.
</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/D29004">View Revision</a></tr></table><br /><div><div><p>If filelight is launched with a path I'm fairly certain it's quicker for the user to just correct the path rather than navigating the open dialog, so go ahead. (I'm almost tempted to just print a warning on the command line).</p>

<p>But this will close the window no matter how the file is opened? Closing the window if the user just selected the wrong path in the open dialog is not good, better to just do it "closer" to where the error happens (I'd suggest nuking the MainWindow::scan(), making MainWindow::slotScanUrl() public, and then call that and quit from main.cpp if that returns false).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R352 Filelight</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29004">https://phabricator.kde.org/D29004</a></div></div><br /><div><strong>To: </strong>shubham, sandsmark, ngraham<br /><strong>Cc: </strong>kde-utils-devel<br /></div>