Baloo KCM text improvements
AbandonedPublic

Authored by ngraham on Nov 10 2017, 11:33 PM.

Details

Reviewers
abetts
Group Reviewers
VDG
Plasma
Summary

BUG: 386634

Implement @abetts' proposed text changes for the Baloo KCM

Test Plan

Tested in KDE Neon:

Diff Detail

Repository
R119 Plasma Desktop
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
ngraham created this revision.Nov 10 2017, 11:33 PM
Restricted Application added a project: Plasma. · View Herald TranscriptNov 10 2017, 11:33 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ngraham edited the test plan for this revision. (Show Details)Nov 10 2017, 11:33 PM
apol added a subscriber: apol.Nov 11 2017, 1:03 AM

Why does the user need to know about the indexing? isn't it an implementation detail?

To be honest I wasn't totally sure about this change either, but we should ask @abetts.

I think that people these days understand that indexing your files is a component of searching. If the word indexing is not liked, maybe it could be

Capture file content data
Read file content data
Record file content data

Etc.

I would also suggest that you start each checkbox with a verb.

I kind of think all of those are worse than what we already have, to be honest. "Do not search in these locations" is a pretty clear and user-friendly text string IMHO.

lueck added a subscriber: lueck.Nov 11 2017, 7:44 AM
lueck added inline comments.
doc/kcontrol/baloo/index.docbook
18–19

Please bump date + releaseinfo

ngraham abandoned this revision.Nov 28 2017, 4:49 AM