Bug 490935

Summary:	System Monitor: Sorting by name is done in the wrong initial order
Product:	[Applications] plasma-systemmonitor	Reporter:	neobrain <bugs.7ng0x>
Component:	general	Assignee:	KSysGuard Developers <ksysguard-bugs>
Status:	RESOLVED FIXED
Severity:	minor	CC:	ahiemstra, nate, plasma-bugs
Priority:	NOR	Keywords:	usability
Version:	6.1.3
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	https://invent.kde.org/plasma/plasma-systemmonitor/-/commit/3ec340a6d787ec5b1bad2c4e2d5ccf96758b7224	Version Fixed In:	6.1.5
Sentry Crash Report:

Description neobrain 2024-07-28 16:27:13 UTC

SUMMARY
When switching to "sort by name" in the Processes view of System Monitor by clicking the corresponding column, processes are sorted from Z-A first. This is counterintuitive and inconsistent with most other applications.

STEPS TO REPRODUCE
1. Open the Processes view in System Monitor
2. Click the Size column to sort by something-that's-not-the-name
3. Click the Name column to sort by process name

OBSERVED RESULT
Processes are sorted from Z to A. The user must click the column again to sort from A to Z.

EXPECTED RESULT
Processes should be sorted from A to Z on first click.

SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 
KDE Frameworks Version: 
Qt Version: 

ADDITIONAL INFORMATION

Comment 1 Nate Graham 2024-07-30 21:09:32 UTC

Can confirm.

Comment 2 Bug Janitor Service 2024-08-06 10:41:43 UTC

A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-systemmonitor/-/merge_requests/304

Comment 3 Arjen Hiemstra 2024-08-09 09:57:51 UTC

Git commit f1e1446eb173f7cd0508f55354ae1d6d43e8a350 by Arjen Hiemstra.
Committed on 06/08/2024 at 10:40.
Pushed by ahiemstra into branch 'master'.

table: Fix default sort order when doing initial sort on a column

Apparently I missed something when porting this to upstream controls, we
don't have a model here anymore.

M  +2    -1    src/table/TableViewHeader.qml

https://invent.kde.org/plasma/plasma-systemmonitor/-/commit/f1e1446eb173f7cd0508f55354ae1d6d43e8a350

Comment 4 Arjen Hiemstra 2024-08-09 09:59:34 UTC

Git commit 3ec340a6d787ec5b1bad2c4e2d5ccf96758b7224 by Arjen Hiemstra.
Committed on 09/08/2024 at 09:58.
Pushed by ahiemstra into branch 'Plasma/6.1'.

table: Fix default sort order when doing initial sort on a column

Apparently I missed something when porting this to upstream controls, we
don't have a model here anymore.


(cherry picked from commit f1e1446eb173f7cd0508f55354ae1d6d43e8a350)

Co-authored-by: Arjen Hiemstra <ahiemstra@heimr.nl>

M  +2    -1    src/table/TableViewHeader.qml

https://invent.kde.org/plasma/plasma-systemmonitor/-/commit/3ec340a6d787ec5b1bad2c4e2d5ccf96758b7224