Bug 389601 - Consider showing star ratings on browse lists and app pages
Summary: Consider showing star ratings on browse lists and app pages
Status: RESOLVED FIXED
Alias: None
Product: Discover
Classification: Applications
Component: discover (show other bugs)
Version: 5.11.95
Platform: Other Linux
: NOR wishlist
Target Milestone: ---
Assignee: Aleix Pol
URL:
Keywords: usability
Depends on:
Blocks:
 
Reported: 2018-01-29 15:14 UTC by Nate Graham
Modified: 2018-04-08 23:23 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In: 5.13.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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