Improve the discoverability of the Filter Text plugin and its menu item
ClosedPublic

Authored by ngraham on Jul 26 2018, 5:13 AM.

Details

Summary

The "Filter Text" plugin is one of Kate's most useful plugins for serious professional use, but it's rather hidden behind a nondescript name, caption, and menu item.

This patch improves the caption to mention "terminal commands" to give a hint about its true power, and also uses a compact variant of that same terminology for the menu item. These changes should improve discoverability.

BUG: 396864
FIXED-IN: 18.12.0

Test Plan

More descriptive caption in the plugins list:

More descriptive menu item text:

Diff Detail

Repository
R40 Kate
Branch
make-filter-text-plugin-more-discoverable (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 1220
Build 1234: arc lint + arc unit
ngraham created this revision.Jul 26 2018, 5:13 AM
Restricted Application added a project: Kate. · View Herald TranscriptJul 26 2018, 5:13 AM
Restricted Application added a subscriber: kwrite-devel. · View Herald Transcript
ngraham requested review of this revision.Jul 26 2018, 5:13 AM
ngraham edited the summary of this revision. (Show Details)Jul 26 2018, 5:15 AM
ngraham edited the test plan for this revision. (Show Details)
lueck added a subscriber: lueck.Jul 26 2018, 6:10 AM

+1

Please adapt the Text Filter plugin in the plugins.docbook https://cgit.kde.org/kate.git/tree/doc/kate/plugins.docbook and
remove the comment "<!--Name is completely misleading and should be changed-->" in line 2887

ngraham updated this revision to Diff 38511.Jul 26 2018, 1:48 PM
ngraham edited the test plan for this revision. (Show Details)

Update docbook accordingly

Restricted Application added a project: Documentation. · View Herald TranscriptJul 26 2018, 1:48 PM
Restricted Application added a subscriber: kde-doc-english. · View Herald Transcript
dhaumann added inline comments.Jul 26 2018, 9:33 PM
doc/kate/plugins.docbook
2884

...be copied... I think a be is missing here.

ngraham marked an inline comment as done.Jul 26 2018, 10:40 PM
ngraham updated this revision to Diff 38536.Jul 26 2018, 10:40 PM

Fix embarrassing error

All good now?

dhaumann accepted this revision.Jul 29 2018, 5:38 AM

Ok with me.

BTW, has the text filter plug-in also have a command line command (F7)?

This revision is now accepted and ready to land.Jul 29 2018, 5:38 AM

Ok with me, too.
A command for F7 would be nice.

cullmann accepted this revision.Jul 29 2018, 10:14 AM
This revision was automatically updated to reflect the committed changes.