Discover in 5.10 supports user reviews, yay! But they're totally buried in the UI, not displayed in a manner that really makes them useful. Two changes are required to make them useful: 1. Display each application's average star rating alongside its icon and description, both in the application list, and the detail view. 2. Display all user reviews underneath the description in the detail view; don't make the user click "Show Comments..." They aren't comments anyway; they're reviews.
Git commit e3eb4dd77df2542fc24b0e2b492712ee7b795543 by Aleix Pol. Committed on 10/07/2017 at 15:46. Pushed by apol into branch 'master'. Use "reviews" rather than "comments" for consistency M +1 -1 discover/qml/ApplicationPage.qml https://commits.kde.org/discover/e3eb4dd77df2542fc24b0e2b492712ee7b795543
The current state was a conscious decision of the current design team. If you want to be part of it, tell me and I'll include you.
I would like to be a part of that discussion, thanks. I feel that hiding the reviews is a mistake because hidden UI elements make sense only in the following scenarios: 1. There is no room to display the UI elements in the main view 2. The hidden UI elements represent infrequently-used advanced functionality or could be confusing to novice users Neither of the following are true here: there is plenty of room to display the reviews, and they are frequently used basic functionality that *improve* the experience for novice users
We all talked in the Discover Design room over Telegram today and I believe the consensus was that we should re-visit the display of reviews and ratings.
Here's an idea: instead of hiding all reviews in a pop-up, we could maybe show the top three reviews in terms of rated helpfulness, and hide the rest under a "show more" link.
author Aleix Pol <aleixpol@kde.org> committer Aleix Pol <aleixpol@kde.org> commit acf66dd1854028d600e4e628d3816e16308a85e6 (patch) tree 17d019c50b4da5ce61450ec7130d0682dadc34e3 parent 116f19e692bb6d3b4bb7243a9c94a87d14da0a2b (diff) Bring back embedded top3 reviews on the ApplicationPageHEADmaster Summary: FEATURE: 380514 Reviewers: ngraham, #discover_software_store Subscribers: ngraham, plasma-devel Tags: #plasma Differential Revision: https://phabricator.kde.org/D10237