<table><tr><td style="">rkflx 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><p>+1 for the concept and a version without the line and the bigger stars everywhere:</p>

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

<p>More comments:</p>

<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>
<li class="remarkup-list-item">As for the horizontal line, I don't think it is necessary, because the stars and the grey rating text already provide some kind of separation between title and the text below.</li>
<li class="remarkup-list-item">I don't like how you made the titles bold in the latest version, I think that's too much. Either bold titles and a regular font size, or no bold titles and the larger font size, but not both.</li>
<li class="remarkup-list-item">Looking at random screenshots of the app store on macOS and GNOME's software center, both don't have such large titles like Discover has. Might be worth for a separate patch to play around with the size a bit, to make the list less screamy and less mobiley.</li>
</ul></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>