<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/D12077">View Revision</a></tr></table><br /><div><div><p>Hm, seems like the width of the places panel is set in <tt style="background: #ebebeb; font-size: 13px;">kdeglobals</tt> (and too small, BTW). After resetting this, without <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Sorting</span></span></span> it's a +1 from my side (did not look at the code, though).</p>
<hr class="remarkup-hr" />
<p>As for <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Sorting</span></span></span>: If a user does not click on the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Configure</span></span></span> button which then has "Sorting" in plain sight, why would he click on a <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Sorting</span></span></span> icon? The problem is not that sorting is too hidden, but that users don't get the idea to actually click on any button in the toolbar. Moving sorting does not help at all with that.</p>
<p>For the view modes it's different, because users recognize the icon from Dolphin. I'd just add the view modes, and IMO sorting in the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Detailed View</span></span></span> is a good enough compromise. We could think about making this mode the default, because the horizontal scrolling of <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Short View</span></span></span> is kinda difficult to use anyway.</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/D12077">https://phabricator.kde.org/D12077</a></div></div><br /><div><strong>To: </strong>ngraham, Frameworks<br /><strong>Cc: </strong>rkflx, mmustac, broulik, michaelh, ngraham, bruns<br /></div>