Created attachment 119282 [details]
screenshot inconsitent sort order
None of the available sort modes allows the order of the file-name-column to match that of the link-column - see screenshot!
STEPS TO REPRODUCE
1. Open folder A with files, that differ in case
2. split screen to second empty folder B
3. create relative links to all files from A in B
4. create more random files in B
5. sort B by link and try to match the order of A which is sorted for filename
allow one config / sort mode to match the order in those two columns
Linux/KDE Plasma: 4.18.0-17-generic
(available in About System)
KDE Plasma Version: 5.13.5
KDE Frameworks Version: 5.50.0
Qt Version: 5.11.1
Only the NameRole gets special treatment, including natural sorting and case-insensitive sorting. At least DestinationRole, but probably also PathRole need the same treatment.
Thinking of it, comments, tags, or other textual roles (genre, artist, album, title, ...) could also benefit from case-insensitive comparison.
I am new to Open Source and I am thinking of getting started with fixing this bug. Any help on getting started would be super helpful.
Hi Madhur, the first step would be to compile and run Dolphin from source on your system. Which packages you need depends on the distribution. Most package managers know how to install these dependencies.
Hi, I am new to Open Source,
So far, I already have my build environment set up and I compiled and ran Dolphin from the source.
I need help in fixing this bug.
Did you read the link in comment #1? If you don't understand the code linked there, please ask specific questions on KDE or Qt developer channels (mailing lists, IRC, etc.)
Made a patch D25741. Although I feel it could take an improvement.
Thank you very much! I'll take a look.
Git commit 9e3418bd558293a92b2e8bcba55f5a3f5d3cc5a4 by Nate Graham, on behalf of George Vogiatzis.
Committed on 28/01/2020 at 15:52.
Pushed by ngraham into branch 'master'.
Add natural sorting and case-insensitive sorting for all role-types
Add natural sorting and case-insensitive sorting, for all role-types
that benefit from.
Sort by any role type specified in `isRoleValueNatural()`
Before: Sorting is always case sensitive
After: Sorting according to 'Sorting mode' in configuration.
Reviewers: #dolphin, nicolasfella, meven, elvisangelaccio, ngraham
Reviewed By: #dolphin, meven, elvisangelaccio, ngraham
Subscribers: cfeck, meven, kfm-devel
Differential Revision: https://phabricator.kde.org/D25741
M +18 -8 src/kitemviews/kfileitemmodel.cpp
M +21 -1 src/kitemviews/kfileitemmodel.h
Thanx. Looking forward for the fix to arrive in the updates.