> On Jan. 19, 2013, 5 p.m., Albert Astals Cid wrote: > > kmenuedit/treeview.h, line 147 > > <http://git.reviewboard.kde.org/r/107409/diff/2/?file=107707#file107707line147> > > > > This should be SortType not and int > > Julien Borderie wrote: > Absolutely, but this slot is called from the QSignalMapper, and I can't > directly connect the signal "mapped(int)" to the slot "sort(SortType)". > A solution should be using an intermediary slot, which calls > "sort(SortType)", but is it not ugly ? :p
Or another solution could be removing the QSignalMapper, and creating as many slots as sorting types (i.e. 4 slots), but we loose benefit of the QSignalMapper. What do you think about this issue ? - Julien ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107409/#review25800 ----------------------------------------------------------- On Jan. 19, 2013, 10:09 p.m., Julien Borderie wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/107409/ > ----------------------------------------------------------- > > (Updated Jan. 19, 2013, 10:09 p.m.) > > > Review request for kde-workspace and Albert Astals Cid. > > > Description > ------- > > Hi, > > This patch adds 2 actions to sort items in KMenuEdit (by name or description) > : > - sort sub-elements for the current selection. > - sort all elements. > > Particularities : > - It respects separator elements to avoid mixing elements groups together. > - Actions are available in the main menu, toolbar and contextual menu. > - Once saved, changes also visible in the K menu. > - Recursive sort. > - Selection sorting is disabled if the selection is empty or is not a menu. > > Thank you for your review. > > > This addresses bug 108419. > http://bugs.kde.org/show_bug.cgi?id=108419 > > > Diffs > ----- > > kmenuedit/kmenuedit.cpp 6a0506b > kmenuedit/kmenueditui.rc 273847d > kmenuedit/main.cpp 65efdf9 > kmenuedit/treeview.h 0284584 > kmenuedit/treeview.cpp b144b1c > > Diff: http://git.reviewboard.kde.org/r/107409/diff/ > > > Testing > ------- > > If started KMenuEdit does not make part of the running KDE version, these > following test cases require to make a kbuildsycoca4 to be visible in the > current K menu. > > 1) Sort element by name > - Select an element > - Choose "sort by name" --> item children sorted > - Now save (and kbuildsycoca4) --> kmenu displays sorted children > > 2) Sort element by description > - Select an element > - Choose "sort by description" --> item children sorted > - Now save (and kbuildsycoca4) --> kmenu displays sorted children > > 3) Sort all by name > - Choose "sort all by name" --> all items sorted > - Now save (and kbuildsycoca4) --> kmenu displays sorted items > > 4) Sort all by description > - Choose "sort all by description" --> all items sorted > - Now save (and kbuildsycoca4) --> kmenu displays sorted items > > > Thanks, > > Julien Borderie > >
