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