-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105778/
-----------------------------------------------------------

Review request for Plasma.


Description
-------

I fail to figure out from the existing code the intention of disabling the 
"Delete" action after deleting one item. Maybe for the edge case of the last 
item being deleted? But the patched code works well in that edge case, too.

The git history does not provide much hint. Those lines of disabling those 
actions were written and last updated almost ten year ago.


This addresses bug 296682.
    http://bugs.kde.org/show_bug.cgi?id=296682


Diffs
-----

  kmenuedit/treeview.cpp d0e6553 

Diff: http://git.reviewboard.kde.org/r/105778/diff/


Testing
-------

I haven't noticed any problem after applying this patch.


Thanks,

Jekyll Wu

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to