search

mpcjanssen / simpletask-android

GNU General Public License v3.0
549 stars 128 forks source link

keep_prio_summary text is irritating, correction suggested #1149

Closed sphenotux closed 2 years ago

sphenotux commented 2 years ago

Standard todotxt behaviour is to remove priority (A), (B), (C) from done tasks, see documentation:

Many Todo.txt clients discard priority on task completion.

(Also linked and discussed here.)

Current behaviour in 10.9.3 depends on settings: When a task with a prio is marked as "done",

IMHO, header of _keepprio is correct. But _keepprio_summary_ of the _keepprio option says:

Todo.txt removes priority on completion

Suggestion: Change removes to keeps. This fits the checkbox state.

Long story short: make _keep_priosummary

Todo.txt keeps priority on completion

PR will follow for English and German language version.

sphenotux commented 2 years ago

Thank you for merging, @mpcjanssen! IMHO this issue may be closed, but translations to other languages than EN+DE open. Conclusion: Simpletask keeps priority on completion