Review Request 107409: [KMenuEdit] Ability to sort elements
Julien Borderie
frajibe at gmail.com
Fri Jan 18 09:27:01 GMT 2013
-----------------------------------------------------------
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.
Changes
-------
remove useless patch.diff
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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130118/dd3cb2f0/attachment.htm>
More information about the kde-core-devel
mailing list