----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128574/#review101146 -----------------------------------------------------------
+1 - Aleix Pol Gonzalez On Aug. 2, 2016, 12:08 p.m., Mathis Beer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128574/ > ----------------------------------------------------------- > > (Updated Aug. 2, 2016, 12:08 p.m.) > > > Review request for KDE Frameworks, Pieter David, David Faure, and Harald > Sitter. > > > Bugs: 343452 > https://bugs.kde.org/show_bug.cgi?id=343452 > > > Repository: kio > > > Description > ------- > > KDirSortFilterProxyModel is advertised in the header as performing a "natural > sort", ie. "7 8 9 10 11", instead of a lexical "10 11 7 8 9". > However, as far as I can tell this was never true from the start, since the > collator responsible for the actual sorting did not have the requisite > numeric mode enabled, and this setting has always been off by default as far > back as I can find docs for it (Qt 5.2). > > (Dolphin, which offers "natural sort", did not run into this issue because it > does not actually use KDirSortFilterProxyModel.) > > > Diffs > ----- > > src/filewidgets/kdirsortfilterproxymodel.cpp 89505ac > > Diff: https://git.reviewboard.kde.org/r/128574/diff/ > > > Testing > ------- > > Create a folder with three images, "File 1.png", "File 10.png", "File 2.png". > View the folder in Gwenview with sort order set to "Name". The sort order is > "File 1.png, File 10.png, File 2.png". > Apply patch. > View the folder in Gwenview with sort order set to "Name". The sort order is > "File 1.png, File 2.png, File 10.png". > > > Thanks, > > Mathis Beer > >
