wtfutil / wtf

The personal information dashboard for your terminal
http://wtfutil.com
Mozilla Public License 2.0
15.83k stars 804 forks source link

Pressing 'q' in text input modal quits the app #1257

Closed lsipii closed 2 years ago

lsipii commented 2 years ago

What's broken?

Text input modal, used in for example Todo-module, quits the whole application when inputting a letter 'q'.

lsipii commented 2 years ago

The q-keypress listerer at wtf_app.go should maybe be a bit more context aware.

Edit: the easiest fix would be to wrap the quit-call with the !wtfApp.focusTracker.IsFocused condition, but that seems to track wheter any module is active. A more useful condition would be one that checks if any text-input is focused.

senorprogrammer commented 2 years ago

v0.42.0 has been released, which fixes this https://github.com/wtfutil/wtf/releases/tag/v0.42.0