[Differential] [Requested Changes] D1955: projectmanagerview: Add Cut command into the context menu

Milian Wolff noreply at phabricator.kde.org
Sat Feb 18 22:44:52 UTC 2017


mwolff requested changes to this revision.
mwolff added a comment.


  this is a really useful feature, I would love to see it fully implemented (i.e. also with `paste`). Can you do that? Or should we take over?

INLINE COMMENTS

> projectmanagerviewplugin.cpp:64
>  
> +static QAction* createSeparatorAction()
> +{

instead of static, wrap it in an anonymous namespace

the result is the same, but I much prefer the C++ way of doing this

REPOSITORY
  R33 KDevPlatform

REVISION DETAIL
  https://phabricator.kde.org/D1955

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: aspotashev, mwolff, kfunk
Cc: mwolff, kfunk, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170218/d542d4ef/attachment.html>


More information about the KDevelop-devel mailing list