D17308: Make CTags/Projects context menu more accessible

Nathaniel Graham noreply at phabricator.kde.org
Sun Dec 2 18:41:51 GMT 2018


ngraham added a comment.


  Oh wow, this would be awesome. This is only visible if the CTags plugin is active, right?
  
  With this change, it might make sense to remove the `CTags: ` on the beginning of the labels.

INLINE COMMENTS

> kate_ctags_view.cpp:66
>      QAction *lookup = actionCollection()->addAction(QLatin1String("ctags_lookup_current"));
> -    lookup->setText(i18n("Lookup Current Text"));
> +    lookup->setText(i18n("Lookup Selected Text"));
>      connect(lookup, &QAction::triggered, this, &KateCTagsView::lookupTag);

While you're touching this user-facing string, let's fix the grammar for all the places where it's currently wrong, because "lookup" is a noun, not a verb.

`Lookup Selected Text` -> `Look Up Selected Text`

REPOSITORY
  R40 Kate

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

To: gregormi, #kate
Cc: ngraham, kwrite-devel, hase, michaelh, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20181202/da6c5d39/attachment.html>


More information about the KWrite-Devel mailing list