-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107409/
-----------------------------------------------------------
(Updated Jan. 18, 2013, 8:22 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 (updated)
-----
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.
File Attachments
----------------
http://git.reviewboard.kde.org/media/uploaded/files/2013/01/17/patch.diff
Thanks,
Julien Borderie
