<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/D12849">View Revision</a></tr></table><br /><div><div><p>Huge +1 on Henrik's suggestion for removing "Vendor Default" in another patch. This one is looking great though!</p>
<p>However, by increasing the vertical size so much, the content is now going to require scrolling at the default System Settings window size of roughly 1024x768 , which rarely looks good in a KCM. Could we try putting each of the sections into a tab within a new top-level tabbed view? This might also help separate out the use cases and user approachability issue, since as Henrik astutely mentioned, the top <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">System Font Selection</span></span></span> section is simple enough to be used by basic users, while the bottom <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Font Rendering</span></span></span> section is quite nerdy and technical. If we do this, we could also remove the titles from the page, and let the tab titles speak for themselves, saving even more vertical space.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>BRANCH</strong><div><div>fonts_kcm_layout (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12849">https://phabricator.kde.org/D12849</a></div></div><br /><div><strong>To: </strong>progwolff, mart, abetts, ngraham<br /><strong>Cc: </strong>zzag, rkflx, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>