D22763: [KDirOperator] Use more human-readable sort order descriptions
Méven Car
noreply at phabricator.kde.org
Sat Jul 27 18:07:01 BST 2019
meven marked an inline comment as done.
meven added inline comments.
INLINE COMMENTS
> ngraham wrote in kdiroperator.cpp:1992
> Why change this? kdiroperator should definitely be ported to use the new connect syntax, but it should be done all at once in its own patch, not one at a time.
_k_slotSortReversed parameter means whether or not the sorting is reversed.
Before this change, the descending action was a check box meaning its value should be the parameter to _k_slotSortReversed.
But now the descending action is a radio button meaning when triggered it should always pass false to _k_slotSortReversed.
A closure was the simple course of action to implement this, and could be reused as is for the ascending action as well.
Onyl the new syntax allows closures to be used as slots.
REPOSITORY
R241 KIO
BRANCH
arcpatch-D22763
REVISION DETAIL
https://phabricator.kde.org/D22763
To: meven, ngraham, #frameworks, nicolasfella
Cc: kde-frameworks-devel, LeGast00n, sbergeron, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190727/1eb40639/attachment.html>
More information about the Kde-frameworks-devel
mailing list