Bug 490935 - System Monitor: Sorting by name is done in the wrong initial order
Summary: System Monitor: Sorting by name is done in the wrong initial order
Status: RESOLVED FIXED
Alias: None
Product: plasma-systemmonitor
Classification: Applications
Component: general (show other bugs)
Version: 6.1.3
Platform: Other Linux
: NOR minor
Target Milestone: ---
Assignee: KSysGuard Developers
URL:
Keywords: usability
Depends on:
Blocks:
 
Reported: 2024-07-28 16:27 UTC by bugs.7ng0x
Modified: 2024-08-09 15:27 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.1.5
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description bugs.7ng0x 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