[KDE Usability] Review Request 126467: Add Case Sensitive Sorting to Dolphin

Emmanuel Pescosta emmanuelpescosta099 at gmail.com
Wed Dec 23 16:42:48 GMT 2015



> On Dec. 22, 2015, 3:47 p.m., Kai Uwe Broulik wrote:
> > settings/general/behaviorsettingspage.cpp, lines 67-68
> > <https://git.reviewboard.kde.org/r/126467/diff/1/?file=425200#file425200line67>
> >
> >     Fix indentation
> 
> arnav dhamija wrote:
>     Thank you for the review. But, on my text editor the indentation looks fine. How may I resolve this issue?
>     
>     EDIT: Fixed, I had accidentaly used the spacebar instead of tab spacing to indent it.

The identation is still wrong, please change the settings of your editor, so that it uses 4 spaces for one tab.


- Emmanuel


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


On Dec. 23, 2015, 8:31 a.m., arnav dhamija wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126467/
> -----------------------------------------------------------
> 
> (Updated Dec. 23, 2015, 8:31 a.m.)
> 
> 
> Review request for Dolphin, KDE Usability and Emmanuel Pescosta.
> 
> 
> Bugs: https://bugs.kde.org/show_bug.cgi?id=148550
>     https://bugs.kde.org/show_bug.cgi?id=https://bugs.kde.org/show_bug.cgi?id=148550
> 
> 
> Repository: dolphin
> 
> 
> Description
> -------
> 
> See also: https://git.reviewboard.kde.org/r/112330/
> 
> This patch was redone because of the transition to KF5.
> 
> This patch adds an option for case sensitive sorting in Dolphin under General Settings by updating the Case Sensitivity of the QCollator in kfileitemmodel.cpp as with the user's selection. Much of what I have done has been based off the code for the Natural Sorting option.
> 
> However I have made a few observations of the results:
> 1) Toggling the Natural Sorting or the Case Sensitive Sorting checkbox in General Settings will not update the sorting of the files in a directory until Dolphin is restarted (I guess this is a bug?).
> 2) If Natural Sorting AND Case Sensitive Sorting are enabled together, Dolphin will sort the files in Natural Sorting using Case Insensitive mode. This is probably a bug on my part - so I would be glad to have a more experienced programmer take a look at it.
> 
> After some thinking, if Case Sensitive Sorting and Natural Sorting cannot play well together, we can reorganise the General Settings option menu as follows:
> 
> <CheckBox>Enable Extra Sorting Features
>         <RadioButton>Natural Sorting
>         <RadioButton>Case Sensitive Sorting
>         
> An example of this can be seen in the screenshot attached with this review request.
> 
> 
> Diffs
> -----
> 
>   settings/general/behaviorsettingspage.h 6e49169 
>   kitemviews/kfileitemmodel.h 167f508 
>   kitemviews/kfileitemmodel.cpp 5f6fed0 
>   settings/general/behaviorsettingspage.cpp 093a1f4 
>   settings/dolphin_generalsettings.kcfg 9ff14d1 
> 
> Diff: https://git.reviewboard.kde.org/r/126467/diff/
> 
> 
> Testing
> -------
> 
> manual
> 
> 
> File Attachments
> ----------------
> 
> Idea for Dolphin General Settings behavior tab
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/12/23/d8e9eab1-1c9e-4cac-84e6-11ad10c077e9__dolphin_settings.png
> 
> 
> Thanks,
> 
> arnav dhamija
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20151223/83f71009/attachment.htm>
-------------- next part --------------
_______________________________________________
kde-usability mailing list
kde-usability at kde.org
https://mail.kde.org/mailman/listinfo/kde-usability


More information about the kfm-devel mailing list