<table><tr><td style="">sitter created this revision.<br />sitter added a reviewer: sandsmark.<br />Restricted Application added a subscriber: kde-utils-devel.
</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/D9285" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">File::name is now File::decodedName</li>
<li class="remarkup-list-item">new File::displayName which properly applies separators to the name (on Windows src/ should be src\)</li>
</ul>

<p>The motivation behind this change is to force us having to make a decision<br />
if we want the low-overhead name or the high-overhead UI name.</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">File::fullPath is now File::displayPath for the same reason as above</li>
<li class="remarkup-list-item">new File::url as a convenience wrapper used by the display* functions to construct a full URL out of their tree so we can determine whether the path is local or remote (scheme file:// vs. http:// etc.)</li>
<li class="remarkup-list-item">RadialMap::Widget::mouseMoveEvent had its tooltip adjusted to not resolve it against the held tree but rather the entities tree. This was inconsistent with hover signals emitted and caused remote paths to get improperly displayed like local paths in the tooltip while being correctly displayed as URLs in the rest of the UI</li>
</ul></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">paths work beautifully on win32</li>
<li class="remarkup-list-item">filelighting over sftp:// also has correct paths now</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R352 Filelight</div></div></div><br /><div><strong>BRANCH</strong><div><div>Applications/17.12</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D9285" rel="noreferrer">https://phabricator.kde.org/D9285</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/fileTree.cpp<br />
src/fileTree.h<br />
src/localLister.cpp<br />
src/radialMap/labels.cpp<br />
src/radialMap/map.cpp<br />
src/radialMap/widget.cpp<br />
src/radialMap/widgetEvents.cpp<br />
src/scan.cpp<br />
src/summaryWidget.cpp</div></div></div><br /><div><strong>To: </strong>sitter, sandsmark<br /><strong>Cc: </strong>kde-utils-devel<br /></div>