<table><tr><td style="">richardbowen 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/D11757">View Revision</a></tr></table><br /><div><div><p>I prefer the system settings as it currently is in 5.12, though the submenu heading could be a little bit darker.</p>
<p>As for the changes made, I think the person that made the main sidebar header color different from the sub-sidebar header color did so intentionally, in line with the menu heirarchy. The horizontal line you added separating the search section from the sidebar options is cool though. The back navigation as it is is pretty clear, no need to turn it into a button.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R124 System Settings</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11757">https://phabricator.kde.org/D11757</a></div></div><br /><div><strong>To: </strong>ngraham, mart, Plasma, VDG<br /><strong>Cc: </strong>richardbowen, plasma-devel, ragreen, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>