D10237: Bring back embedded top3 reviews on the ApplicationPage

[Nathaniel Graham]

ngraham requested changes to this revision.
ngraham added a comment.
This revision now requires changes to proceed.


  F5687564: What it looks like.png <https://phabricator.kde.org/F5687564>
  
  Nice! There are a few polish changes I'd like to see:
  
  - Add a header label saying "Top reviews", since it's otherwise not obvious what users are looking at ("Newest reviews?" "Developer's favorite reviews?"). We could probably replace the blue line with this header
  - Change the label that says "Show Reviews" to "Show more reviews" (unless you deliberately avoided changing any strings to squeak it in for 5.12)
  - Could we increase the vertical padding between each review a bit?
  - For KDenlive's entry, I noticed three extra dummy reviews:
  
  F5687562: Extra reviews.png <https://phabricator.kde.org/F5687562>

REPOSITORY
  R134 Discover Software Store

REVISION DETAIL
  https://phabricator.kde.org/D10237

To: apol, ngraham
Cc: ngraham, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180201/3d53dc09/attachment.html>