<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/D11876">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/D11876#241864" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D11876#241864</a>, <a href="https://phabricator.kde.org/p/rkflx/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@rkflx</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><ul class="remarkup-list">
<li class="remarkup-list-item">Having the stars roughly the same height as the text next to it (like in the screenshot above, from an earlier iteration) works good enough for me. You might want to check how this looks with the default fonts size, though, because I suspect you bumped yours up a bit. Or could both heights be linked together?</li>
</ul></div>
</blockquote>

<p>The star height uses the point size of the header, currently. It only <em>looks</em> like it's linked to the label size. If we want to link it to the label size, there would need to be some more work done (in another patch, obviously) regarding how the star size is calculated.</p></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11876">https://phabricator.kde.org/D11876</a></div></div><br /><div><strong>To: </strong>ngraham, apol, Discover Software Store, VDG<br /><strong>Cc: </strong>rkflx, fabianr, ragreen, acrouthamel, plasma-devel, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>