<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/D11757">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D11757#235700" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D11757#235700</a>, <a href="https://phabricator.kde.org/p/richardbowen/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@richardbowen</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><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.</p></div>
</blockquote>

<p>That parson was <a href="https://phabricator.kde.org/p/mart/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@mart</a>, which is why I added him as a reviewer. Previously, the light subcategory header text was visually consistent with the category headers on the main list--which were far too light due to a bug. I fixed that bug, and now the subcategory header text no longer matches with anything. It's also inconsistent with other column-based navigation headers, and is just too light in general. 30% opacity text is hard to read and doesn't make conceptual sense for a header.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>The horizontal line you added separating the search section from the sidebar options is cool though.</p></blockquote>

<p>Thanks!</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>The back navigation as it is is pretty clear, no need to turn it into a button.</p></blockquote>

<p>It's not necessarily about clarity, it's about visual polish. Did you read the linked bug? It looks especially weird in HiDPI mode. If we're going to keep the present appearance, we should remove the button borders that show up on hover that are jammed up against the parent frame. But I kind of like bit big explicit button approach myself (obviously, or I wouldn't have proposed it! :-) )</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>