ngraham added a comment.

  In D11876#241864 <https://phabricator.kde.org/D11876#241864>, @rkflx wrote:
  
  > - 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?
  
  
  The star height uses the point size of the header, currently. It only 
//looks// 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.

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

To: ngraham, apol, #discover_software_store, #vdg
Cc: rkflx, fabianr, ragreen, acrouthamel, plasma-devel, Pitel, ZrenBot, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

Reply via email to