Make plugin titles and descriptions consistent in casing and wording
ClosedPublic

Authored by gregormi on Nov 3 2018, 6:27 PM.

Details

Summary

Before:

After:

Note:

Diff Detail

Repository
R40 Kate
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
gregormi created this revision.Nov 3 2018, 6:27 PM
Restricted Application added a project: Kate. · View Herald TranscriptNov 3 2018, 6:27 PM
Restricted Application added a subscriber: kwrite-devel. · View Herald Transcript
gregormi requested review of this revision.Nov 3 2018, 6:27 PM
gregormi edited the summary of this revision. (Show Details)Nov 3 2018, 6:28 PM
gregormi edited the summary of this revision. (Show Details)Nov 3 2018, 6:36 PM
gregormi added a reviewer: Kate.
ngraham added a subscriber: ngraham.Nov 3 2018, 6:37 PM

Just a few wording and grammar suggestions:

addons/filetree/katefiletreeplugin.desktop
5

Maybe "Document Tree View"?

addons/project/kateprojectplugin.desktop
58

How about:

Integration with Git and other source control systems

addons/search/katesearch.desktop
60

folders or -> folders, or (use that Oxford Comma! See https://en.wikipedia.org/wiki/Serial_comma)

Good idea to unify this. Can we have one more iteration? :)

addons/konsole/katekonsoleplugin.desktop
61

I have never heard the wording "at the ready" before...

addons/project/kateprojectplugin.desktop
58

Yes, +1 for Nate's suggestion

FWIW "at the ready" is perfectly correct English. :)

Good idea to unify this. Can we have one more iteration? :)

Yes, sure. I'll deliver :-)

addons/konsole/katekonsoleplugin.desktop
61

I know this term from Harry Potter: https://www.youtube.com/watch?v=IMsd-TAUqv8 ("wands at the ready"). And for me, the power of the Linux command line (when you know how to use it) has some similarities to a wand. :-)

addons/project/kateprojectplugin.desktop
58

I tried to convey that the project plugin can to more than just to integrate with git etc. But in the end I also think Nate's suggestion is easier to grasp.

gregormi updated this revision to Diff 45289.Nov 11 2018, 2:41 PM
  • Apply review comments
gregormi edited the summary of this revision. (Show Details)Nov 11 2018, 2:42 PM
gregormi edited the summary of this revision. (Show Details)Nov 11 2018, 2:45 PM
ngraham accepted this revision.Nov 11 2018, 4:18 PM
ngraham added a reviewer: VDG.

Looks great to me!

This revision is now accepted and ready to land.Nov 11 2018, 4:18 PM
This revision was automatically updated to reflect the committed changes.