<table><tr><td style="">dfaure 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/D27951">View Revision</a></tr></table><br /><div><div><p>OK, I looked at the code more closely and I see now that on different partitions it will still show the menu, rather than make any assumptions.</p>

<p>This still creates a risk for surprises, as to whether the menu will appear or not (on Windows one can look at two paths and use the driver letters to see if it's the same partition or not, that's not possible on Unix). But I see how we can assume that a power user knows where his partitions are. And they can turn off the advanced feature again if they get it wrong too many times.</p>

<p>So I won't veto this.</p>

<p>Implementation note: the fastest way to know if two items are on the same partition is to compare KIO::UDSEntry::UDS_DEVICE_ID values. But unfortunately it looks like this code takes URLs, not proper KFileItems created from a kio_file job... I guess that means statbuf.st_dev directly, on Unix?</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/D27951">https://phabricator.kde.org/D27951</a></div></div><br /><div><strong>To: </strong>trmdi, ngraham, dfaure, meven, VDG, davidedmundson<br /><strong>Cc: </strong>elvisangelaccio, davidedmundson, meven, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns<br /></div>