<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/D20908">View Revision</a></tr></table><br /><div><div><p>Yeah I think keeping the icons on top is nice here since it makes for large click areas. Putting the icon beside the text makes more sense when the number of categories is very large, which isn't normally the case for settings windows.</p>

<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/D20908#458428" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D20908#458428</a>, <a href="https://phabricator.kde.org/p/filipf/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@filipf</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>BTW implementation wise here's one reason why what I did here is not a good solution:</p>

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

<p>Looks like the rectangle needs to not be visible when there's only one category. Maybe we can get that information from <tt style="background: #ebebeb; font-size: 13px;">globalConfigModel</tt>.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20908">https://phabricator.kde.org/D20908</a></div></div><br /><div><strong>To: </strong>filipf, VDG, Plasma, ngraham<br /><strong>Cc: </strong>broulik, GB_2, hein, plasma-devel, jraleigh, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>