--===============8569867708929795800== Content-Type: multipart/alternative; boundary="===============6238539051149345262==" --===============6238539051149345262== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable ----------------------------------------------------------- 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 las= t item being deleted? But the patched code works well in that edge case, to= o. 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=3D296682 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 --===============6238539051149345262== Content-Type: text/html; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable
This is an automatically generated e-mail. To reply, visit: http://git.revie= wboard.kde.org/r/105778/ |
Review request for Plasma.
By Jekyll Wu.
Descripti= on
Testing <= /h1>
Bugs:
296682
Diffs=
|