| Summary: | GridViewKCM: sorting content by name should be case-insensitive | ||
|---|---|---|---|
| Product: | [Applications] systemsettings | Reporter: | Patrick Silva <bugseforuns> |
| Component: | general | Assignee: | Plasma Bugs List <plasma-bugs> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | nate, plasma-bugs |
| Priority: | NOR | ||
| Version: | 5.18.5 | ||
| Target Milestone: | --- | ||
| Platform: | Arch Linux | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed In: | 5.22 | |
| Sentry Crash Report: | |||
| Attachments: | screenshot | ||
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. I have submitted merge requests to fix this for all grid view KCMs: - https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/709 - https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/379 - https://invent.kde.org/plasma/kwin/-/merge_requests/768 - https://invent.kde.org/plasma/plymouth-kcm/-/merge_requests/7 - https://invent.kde.org/plasma/sddm-kcm/-/merge_requests/9 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 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 Fixed in Plasma 5.22 with the combination of those commits. |
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