Bug 389601

Summary: Consider showing star ratings on browse lists and app pages
Product: [Applications] Discover Reporter: Nate Graham <nate>
Component: discoverAssignee: Aleix Pol <aleixpol>
Status: RESOLVED FIXED    
Severity: wishlist Keywords: usability
Priority: NOR    
Version: 5.11.95   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In: 5.13.0

Description Nate Graham 2018-01-29 15:14:45 UTC
Not showing the overall star ratings for apps has always been a minor source of confusion for me, and a major linux distro reviewer objected to this as well in his review of the 5.12 beta: http://www.ocsmag.com/2018/01/27/plasma-5-12-long-term-sweetness/

"For instance, if you go into any which program, to see additional details, you get [...] no rating, even though there’s a link to program reviews."

We should consider exposing the star rating.
Comment 1 Nate Graham 2018-04-01 23:33:26 UTC
Submitted a patch for this: https://phabricator.kde.org/D11876
Comment 2 Nate Graham 2018-04-08 23:23:11 UTC
Git commit 19dbc556dbf29ca3ed804de77aa9cc53d4beeb16 by Nathaniel Graham.
Committed on 08/04/2018 at 23:23.
Pushed by ngraham into branch 'master'.

Display star ratings

Summary:
For quite some time, users and reviewers have been asking for Discover to display star ratings. This is really useful information, and becomes especially important now that we have multiple sort modes. People will think, "this list may be sorted by rating, but what **ARE** the ratings?"

This patch adds star ratings to the browse and search lists, and the detailed view pages.

It also adjusts the look of the delegates to better accommodate stars without becoming cluttered.
FIXED-IN: 5.13.0

Depends on D11842, because we need the extra space gained by removing the less-useful categories.

Test Plan:
Before:
{F5799814}

After, Applications page:
{F5800859}

After, Installed page:
{F5800867}

After, Applications Page in compact or mobile view:
{F5800861}

Reviewers: apol, #discover_software_store, #vdg

Reviewed By: apol, #discover_software_store

Subscribers: abetts, rkflx, fabianr, ragreen, acrouthamel, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D11876

M  +16   -7    discover/qml/ApplicationDelegate.qml
M  +12   -0    discover/qml/ApplicationPage.qml
M  +2    -0    discover/qml/ApplicationsListPage.qml
M  +1    -0    discover/qml/InstalledPage.qml

https://commits.kde.org/discover/19dbc556dbf29ca3ed804de77aa9cc53d4beeb16