<table><tr><td style="">rkflx 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/D10245" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>-1 from my side because:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Dolphin: By default, the sidebar now has an ugly scrollbar. (The screenshot is "staged" in that regard.)</li>
<li class="remarkup-list-item">Gwenview: This clutters the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Places</span></span></span> list on the Start Page with a bunch of stuff not relevant for users (e.g. Music). I'm still busy fixing the fallout from the last patch similarly adding unwanted/broken entries, so please help out if you must add this globally. Note this is not only about Gwenview, but a lot of other apps using <tt style="background: #ebebeb; font-size: 13px;">KFilePlaces</tt> are not interested in some of the entries either.</li>
<li class="remarkup-list-item">In general: Now there's duplication with <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Search For</span></span></span>. For example, what's the difference between "Documents" and "Documents"? (Not everybody is aware of the headers, and rightfully shouldn't!) Same for "Pictures" vs. "Images". Maybe I could live with that if this only appeared if Baloo was turned off.</li>
</ul>

<p>I'd rather use <tt style="background: #ebebeb; font-size: 13px;">KFilePlaces</tt> more as a bookmark thingy, i.e. something where users add custom entries they want to see either in all or in specific apps. Those are highly specialized folders in my experience, depending on user's workflows and not something we could predict. Better leave some space so users can customize themselves.</p>

<p>Sorry Nate, I know you meant well. Nevertheless, let's hear first what others have to say, maybe I'm wrong or we could tone down the list a bit…</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/D10245" rel="noreferrer">https://phabricator.kde.org/D10245</a></div></div><br /><div><strong>To: </strong>ngraham, Frameworks, Dolphin<br /><strong>Cc: </strong>rkflx, michaelh, ngraham<br /></div>