Bug 389601

Summary:	Consider showing star ratings on browse lists and app pages
Product:	[Applications] Discover	Reporter:	Nate Graham <nate>
Component:	discover	Assignee:	Aleix Pol <aleixpol>
Status:	RESOLVED FIXED
Severity:	wishlist	Keywords:	usability
Priority:	NOR
Version First Reported In:	5.11.95
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	https://commits.kde.org/discover/19dbc556dbf29ca3ed804de77aa9cc53d4beeb16	Version Fixed/Implemented In:	5.13.0
Sentry Crash Report:

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