https://bugs.kde.org/show_bug.cgi?id=513139
Akseli Lahtinen <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas |ma/discover/-/commit/9ec868 |ma/discover/-/commit/1703aa |3c094acced024a3e1b71be2814f |7b2811610edc014f68c3544fc4b |674fda3 |763cc42 --- Comment #7 from Akseli Lahtinen <[email protected]> --- Git commit 1703aa7b2811610edc014f68c3544fc4b763cc42 by Akseli Lahtinen. Committed on 18/02/2026 at 14:21. Pushed by akselmo into branch 'Plasma/6.6'. ReviewStats, Rating: Expose starCounts from rating and use it for ReviewStats ReviewStats was calculating the rating progressBars from reviews only, not from all the total reviews. This was confusing, since the actual rating value was calculated from the total ratings. This exposes the star counts in Rating class and changes ReviewStats to utilize that instead, so we get accurate progressbars per rating. Also changes the sorting button name of "Rating" to "Popularity" (cherry picked from commit 9ec8683c094acced024a3e1b71be2814f674fda3) c26733f3 ReviewStats, Rating: Expose starCounts from rating and use it for ReviewStats ec0b6acd ReviewsModel: remove StarsCount 2235dee5 Add comments to Rating 88dbea3c set globalrating to use rating instead of sortableRating 330351f0 Use regular rating for visuals 732d2feb Set starcounts to const std::vector<int> c38a6020 Use popularity instead of rating for sort text Co-authored-by: Akseli Lahtinen <[email protected]> M +1 -1 discover/qml/ApplicationDelegate.qml M +1 -1 discover/qml/ApplicationPage.qml M +1 -1 discover/qml/ApplicationsListPage.qml M +5 -5 discover/qml/ReviewsStats.qml M +7 -1 libdiscover/ReviewsBackend/Rating.cpp M +11 -0 libdiscover/ReviewsBackend/Rating.h M +0 -67 libdiscover/ReviewsBackend/ReviewsModel.cpp M +0 -29 libdiscover/ReviewsBackend/ReviewsModel.h https://invent.kde.org/plasma/discover/-/commit/1703aa7b2811610edc014f68c3544fc4b763cc42 -- You are receiving this mail because: You are watching all bug changes.
