> On Jan. 17, 2013, 8:12 p.m., Christoph Feck wrote: > > Thanks Julien for the patch. Please ask for help in in #kde-devel irc > > channel, or on an KDE developers mailing list. The diff needs to be > > uploaded differently.
Thank you for your reply, it should be correctly uploaded now. - Julien ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107409/#review25734 ----------------------------------------------------------- On Jan. 18, 2013, 9:27 a.m., Julien Borderie wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/107409/ > ----------------------------------------------------------- > > (Updated Jan. 18, 2013, 9:27 a.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. > > It respects separator elements to avoid mixing elements groups together. > Actions are visible in the main menu, toolbar and contextual 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 > ------- > > - Select an element and choose "sort by name". > - Select an element and choose "sort by description". > - Choose "sort all by name". > - Choose "sort all by description". > - Select an empty menu or a simple element -> selection sorting disabled. > > > Thanks, > > Julien Borderie > >
