Bug 470614 - App list page sort order is reversed
Summary: App list page sort order is reversed
Status: RESOLVED FIXED
Alias: None
Product: Discover
Classification: Applications
Component: discover (show other bugs)
Version: master
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords: qt6
Depends on:
Blocks:
 
Reported: 2023-06-03 21:36 UTC by Nate Graham
Modified: 2023-10-06 02:09 UTC (History)
2 users (show)

See Also:
Latest Commit: https://invent.kde.org/plasma/discover/-/commit/d1811594d8352e8059087b9c84d224243711cc81
Version Fixed In:

Attachments
Sort order reversed (598.79 KB, image/jpeg)
2023-06-05 03:37 UTC, Nate Graham 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Nate Graham 2023-06-03 21:36:53 UTC 
See attached screenshot. The apps shown on top should be on the bottom. If I scroll aaaaall the way to the bottom, I see the apps I expect to be on top.
Comment 1 Justin Zobel 2023-06-05 03:17:11 UTC 
No attachment :)
Comment 2 Nate Graham 2023-06-05 03:37:03 UTC 
Created attachment 159468 [details]
Sort order reversed

Derp
Comment 3 Justin Zobel 2023-06-22 14:48:57 UTC 
Can confirm.
Comment 4 Nate Graham 2023-10-06 02:09:48 UTC 
Git commit d1811594d8352e8059087b9c84d224243711cc81 by Nate Graham, on behalf of Amy Rose.
Committed on 06/10/2023 at 04:09.
Pushed by ngraham into branch 'master'.

ResourcesProxyModel: respect sort order

Seems like the sorting done in `ResourcesProxyModel` was never
considering the `Qt::SortOrder`. This commit adds an `orderedLessThan`
method to sort in reverse when appropriate.

M  +9    -3    libdiscover/resources/ResourcesProxyModel.cpp
M  +1    -0    libdiscover/resources/ResourcesProxyModel.h

https://invent.kde.org/plasma/discover/-/commit/d1811594d8352e8059087b9c84d224243711cc81