Patch to sort the tag-values alphabetically in the "Tags" column of Dolphin.
Review Request #130068 - Created April 7, 2017 and submitted
Up until now tag-values have appeared unsorted in the Tags column
when the selected View Mode is "Details".
In older versions of Dolphin (in KDE4) the tags-values were
alphabetically sorted in the Tags column, which means that back then
this was the desired behavior.
This commit restores this functionality.
Related to the following commit:
I built dolphin on my "KDE Neon Developer Edition" environment having applied this patch. No problems appeared during the build.
The tag-values now appear sorted alphabetically in the "Tags" column of Dolphin.
No further testing done.
The naming is wrong. As you can see in the rest of the code, the "camelCase" principle is used, not "camel_case".
This is probably not an "alphabetical order" sort, even though it might seem OK in most cases.
This is the machine sort. So for instance: 9, 3, 5 will become:
That is "alphabetical", but only because there is no number bigger than 9. This: 9, 3, 10, 5
That is not alphabetical.
You need to use QCollator and QCollatorSortKey to get real sorting that we - humans - call natural :)
What you did is likely oke in most cases. Till someone decides to make tags with numbers in them, then you'd hit the issue i just described.
Regarding your pull request. I doubt that is useful on github since it is the KDE mirror. The "live" version lives on cgit.kde.org.
I don't really know how that is suppoed to be handled though.