<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: Plasma, davidedmundson, franckarrecot, ngraham.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.<br />broulik requested review of this revision.
</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/D13841">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p><tt style="background: #ebebeb; font-size: 13px;">KFilePlacesModel</tt> internally only stores <tt style="background: #ebebeb; font-size: 13px;">search:/</tt> URLs and then translates them to the correct Baloo query as needed.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>It's still a bit awkward that you get two "Pictures" results, one being the "Search for Pictures" and the other the "Pictures" folder but at least you can open both of them correctly now.</p>

<p>I'll see if I can maybe add the group name as subText so it looks like "Pictures (Search For)", but it might also add "Places: Pictures (Places)" which is a bit weird</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13841">https://phabricator.kde.org/D13841</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>runners/places/placesrunner.cpp</div></div></div><br /><div><strong>To: </strong>broulik, Plasma, davidedmundson, franckarrecot, ngraham<br /><strong>Cc: </strong>plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>