> On Feb. 10, 2013, 10:54 p.m., Albert Astals Cid wrote: > > kmenuedit/treeview.cpp, line 1562 > > <http://git.reviewboard.kde.org/r/107409/diff/5/?file=111140#file111140line1562> > > > > Do we need all this new function for the "Use name if description is > > empty" thing? Wouldn't it be easier to just make itemDescriptionLessThan > > call itemNameLessThan if both descriptions are emtpy?
Huh... I don't know how I missed that, you're clearly right. Diff uploaded. - Julien ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107409/#review27152 ----------------------------------------------------------- On Jan. 30, 2013, 10:40 p.m., Julien Borderie wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/107409/ > ----------------------------------------------------------- > > (Updated Jan. 30, 2013, 10:40 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/treeview.h 0284584 > kmenuedit/main.cpp 65efdf9 > kmenuedit/kmenueditui.rc 273847d > kmenuedit/kmenuedit.cpp 6a0506b > 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 > >
