Review Request 124795: Provide deterministic sort order for KDirSortFilterProxyModelPrivate::compare

[Albert Astals Cid]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124795/
-----------------------------------------------------------

(Updated Aug. 17, 2015, 9:58 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks and David Faure.


Changes
-------

Submitted with commit 6bd3208c0833236e2a81f3e1213c84f0a0f593fc by Albert Astals Cid to branch master.


Repository: kio


Description
-------

We already provide a deterministic sort order for KDirSortFilterProxyModelPrivate::compare when not using the QCollator, but when using it we don't. This piggy-backs on the existing code to fix that.


Diffs
-----

  src/filewidgets/kdirsortfilterproxymodel.cpp 22ac025 

Diff: https://git.reviewboard.kde.org/r/124795/diff/


Testing
-------

The open dialog doesn't switch like crazy now when hovering between two folders with the same name and different case.


Thanks,

Albert Astals Cid

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150817/6c48612d/attachment-0001.html>