Bug 404608 - GridViewKCM: sorting content by name should be case-insensitive
Summary: GridViewKCM: sorting content by name should be case-insensitive
Status: RESOLVED FIXED
Alias: None
Product: systemsettings
Classification: Applications
Component: general (show other bugs)
Version: 5.18.5
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-02-20 16:55 UTC by Patrick Silva
Modified: 2021-04-09 19:36 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.22


Attachments
screenshot (185.72 KB, image/png)
2019-02-20 16:55 UTC, Patrick Silva
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Silva 2019-02-20 16:55:55 UTC
Created attachment 118218 [details]
screenshot

SUMMARY
See my screenshot please. I have several icon themes, ePapirus is the last one on the list bacause case-sensitive sorting.
Case-insensitive sorting would be nice.


SOFTWARE/OS VERSIONS
Operating System: Arch Linux 
KDE Plasma Version: 5.15.1
KDE Frameworks Version: 5.55.0
Qt Version: 5.12.1
Comment 1 Nate Graham 2021-03-11 00:16:24 UTC
Looking through the code, the common grid view UI component doesn't do any sorting at all, and it is not feasible to put a sortProxyModel into the view itself to make this happen automatically for all data sets. :( So this will need to be implemented for each individual KCM as needed, either in the data model, or with a SortFilterProxyModel on the front-end to change the sorting.

I will do what I can.
Comment 3 Alexander Lohnau 2021-04-09 06:16:51 UTC
Git commit 846a9636fb89df70380ef3d181dc837f36837481 by Alexander Lohnau.
Committed on 09/04/2021 at 06:16.
Pushed by alex into branch 'master'.

ksplash kcm: Sort grid view KCM case insensitively

Since this KCM can display user and distro provided content, we can't
ensure that everything will begin with a capital letter. Accordingly, we
should sort the grid case-insensitively to prevent the entries starting
with lowercase letters from being shunted to the end.

M  +14   -3    kcms/ksplash/kcm.cpp
M  +6    -3    kcms/ksplash/kcm.h
M  +1    -1    kcms/ksplash/package/contents/ui/main.qml

https://invent.kde.org/plasma/plasma-desktop/commit/846a9636fb89df70380ef3d181dc837f36837481
Comment 4 Nate Graham 2021-04-09 19:35:19 UTC
Git commit ae63802acb5270cbb3e4ab72f414d11ffecce132 by Nate Graham.
Committed on 09/04/2021 at 19:35.
Pushed by ngraham into branch 'master'.

[kcms] Sort grid view KCMs case insensitively

Since these KCMs can display user and distro provided content, we can't
ensure that everything will begin with a capital letter. Accordingly, we
should sort the grid case-insensitively to prevent the entries starting
with lowercase letters from being shunted to the end.

M  +2    -0    kcms/colors/colorsmodel.cpp
M  +2    -1    kcms/cursortheme/kcmcursortheme.cpp
M  +1    -1    kcms/cursortheme/xcursor/sortproxymodel.cpp
M  +2    -0    kcms/desktoptheme/themesmodel.cpp
M  +6    -2    kcms/icons/iconsmodel.cpp
M  +9    -2    kcms/lookandfeel/kcm.cpp
M  +2    -0    kcms/style/stylesmodel.cpp

https://invent.kde.org/plasma/plasma-workspace/commit/ae63802acb5270cbb3e4ab72f414d11ffecce132
Comment 5 Nate Graham 2021-04-09 19:36:33 UTC
Fixed in Plasma 5.22 with the combination of those commits.