<table><tr><td style="">ngraham 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>The scrollbar issue is real, though I don't think scrollbars are as awful as you do. :-) Something to think about. I could see removing Music and Pictures, maybe. But Desktop, Documents, and Downloads are heavily used by ordinary regular users and I think it makes sense to provide links to them in the Places panel by default.</p>

<p>This raises a greater question of whether or not it makes sense to have <tt style="background: #ebebeb; font-size: 13px;">KFilePlaces</tt> in Gwenview. Gwenview is an image viewer, and necessarily, the majority of <tt style="background: #ebebeb; font-size: 13px;">KFilePlaces</tt> items are not going to be relevant to pictures. Same for other apps using it. Even without my patch, this issue will occur for users who add their own items to the Places panel.</p>

<p>I feel like the label duplication with the Search For section is a minor issue, and mostly a red herring. The header is right there, so at least in English, the implied imperative sentence is quite obvious: "Search for Documents", "Search for Music," etc. The functions are totally different, and equally important (and if anything, the functionality gained by this patch is more important).</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>