Bug 470614

Summary:	App list page sort order is reversed
Product:	[Applications] Discover	Reporter:	Nate Graham <nate>
Component:	discover	Assignee:	Plasma Bugs List <plasma-bugs>
Status:	RESOLVED FIXED
Severity:	normal	CC:	aleixpol, justin.zobel
Priority:	NOR	Keywords:	qt6
Version:	master
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	https://invent.kde.org/plasma/discover/-/commit/d1811594d8352e8059087b9c84d224243711cc81	Version Fixed In:
Attachments:	Sort order reversed

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