search

vonshednob / pter

Manage your todo.txt in a commandline user interface (TUI)
https://vonshednob.cc/pter/
MIT License
102 stars 6 forks source link

A standalone "+" character is offered as a completion item #46

Closed andrei-a-papou closed 6 months ago

andrei-a-papou commented 6 months ago

Here's a minor bug: when you have a todo like Implement Ctrl + Backspace, pter picks up the plus character as a project and offers it as an autocomplete suggestion. This doesn't happen for the "Select project" dialog though.

Same thing for a standalone '@' char.

andrei-a-papou commented 6 months ago

Here's a quick and dirty fix, couldn't think of a better one :)

98-fix-completion.diff.gz

vonshednob commented 6 months ago

There is exactly one quick and dirty fix I can think of, let’s see if we thought the same :grin:

vonshednob commented 6 months ago

Almost the same. Unfortunately the patch was also triggering crashes when there was no + or @ when completing a + or @. Anyway, it's fixed with this commit!

andrei-a-papou commented 6 months ago

Seems to work great!