PlainTasks icon indicating copy to clipboard operation
PlainTasks copied to clipboard

"done" and "cancelled" tags (from key bindings) have spaces

Open FichteFoll opened this issue 7 years ago • 2 comments

When cancelling or checking a task with a key binding, the inserted tags have spaces between the tag and the timestamp. This is not the case when completing the d snippet for @due or t for @toggle, which makes it inconsistent.

Example: @cancelled (2017-01-21 03:41)

Please remove these spaces.

FichteFoll avatar Jan 21 '17 02:01 FichteFoll

There is undocumented setting before_date_space (must be string). The reason is that initially PT had mandatory whitespace (current default), then we introduced Taskpaper compatibility (mandatory absence of whitespace), other tags were introduced later (and whitespaces question was out of any concerns, because it is very minor and those tags are quite different).

So I don’t know… I’d rather be conservative and won’t touch anything, esp. since you can just add "before_date_space": "" in your settings.

vovkkk avatar Jan 21 '17 03:01 vovkkk

Thanks, that setting does it for me.

It is kind of suboptimal that the tags are inconsistent by default, but I can also see how users would like to have them consistent in their docs, so prompty changing the default setting is also not optimal. There would be ways for migration, but nothing I'd personally want to deal with (i.e. change the default value and add the current default to the User preferences on upgrade but not on install).

FichteFoll avatar Jan 22 '17 15:01 FichteFoll