<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>I brought this up before, but let me repeat: It is of utmost importance to test with the default settings, i.e. default font size and default dialog size (remove <tt style="background: #ebebeb; font-size: 13px;">~/.config/kate*</tt> for that). Your current version does not look good, despite what you show in your screenshot in the summary:</p>

<p><a href="https://phabricator.kde.org/F5803932" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F5803932: kio-toolbar.png</a></p>

<p>Note that I'm not saying your intention in this patch is wrong. But as there is only so much space available, you need to prioritize what can be shown. By default, there should be no overflow arrow in the toolbar too. Ideas:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item"><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> is irritating, this needs an icon, e.g. <tt style="background: #ebebeb; font-size: 13px;">view-sort-descending</tt>, <tt style="background: #ebebeb; font-size: 13px;">object-order-back</tt> or an entirely new icon. OTOH I wonder if this entry is needed at all, because for <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> you could simply click on the table headers.</li>
<li class="remarkup-list-item">The slider should get a minimum size.</li>
<li class="remarkup-list-item">There should still be a good amount of whitespace between what's on the left and on the right side of the toolbar. This separation into groups results in faster navigation in the UI and helps in making it feel less crowded.</li>
</ul>

<p>Please resist the urge to put everything in the toolbar. Configuring the dialog is <em>not</em> a regular task, so using 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 is fine too. Only the most important settings should be shown.</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>