<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/D10237" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Getting there!</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Change the text "Show reviews" to "Show more reviews"</li>
<li class="remarkup-list-item">Maybe change the "Reviews" header to "Top Reviews". Judgment call, not necessarily required</li>
<li class="remarkup-list-item">The increased padding between the reviews is good, but now the divider line is misplaced. It looks really weird not being equidistant between each review. If we can't change that easily, I would advocate getting rid of the line entirely, as the padding is now sufficient to visually separate the reviews.</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R134 Discover Software Store</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10237" rel="noreferrer">https://phabricator.kde.org/D10237</a></div></div><br /><div><strong>To: </strong>apol, ngraham, Discover Software Store<br /><strong>Cc: </strong>ngraham, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>