D4988: Fix for kdevelop component names displayed in the shortcut editor

Sven Brauch noreply at phabricator.kde.org
Thu Mar 9 15:51:32 UTC 2017


brauch accepted this revision.
brauch added a comment.
This revision is now accepted and ready to land.


  LGTM, thanks

INLINE COMMENTS

> clangsupport.cpp:279
>      _xmlFile = xmlFile();
> +    actions.setComponentDisplayName(i18n("Clang Support"));
>  

I think we should present the clang plugin as "C++ language support" or so to the user. "clang support" sounds like support for a compiler which it isn't.

REPOSITORY
  R32 KDevelop

BRANCH
  master

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

To: antonanikin, #kdevelop, brauch
Cc: brauch, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170309/2980bb1c/attachment-0001.html>


More information about the KDevelop-devel mailing list