search

cointop-sh / cointop

A fast and lightweight interactive terminal based UI application for tracking cryptocurrencies 🚀
https://cointop.sh
Apache License 2.0
3.98k stars 311 forks source link

Cannot delete shortcut #240

Closed matthiasbeyer closed 2 years ago

matthiasbeyer commented 2 years ago

I constantly fat-finger q or Q and quit cointop, but I don't want to.

Somehow, though, removing the key bindings from the configuration file does not work, because cointop re-adds them automatically.

Is this a bug? Or is there a way to work around that?

lyricnz commented 2 years ago

Hi, and thanks for your contribution. You're partly running into a recent issue #233, which was fixed a couple of days ago (not in a release yet).

Once you get a version with that fix, there's still kindof an issue where cointop loads all the defaults, then loads your configuration on top, which results in a "merge" which puts any deleted key bindings back. You could just bind q to something innocuous like "save"?

If you can checkout and build source code, you can be on the latest code versions with all the current fixes - and help us with development/testing (I see you have submitted a few other feature requests). Better still if you can contribute fixes! (I'm a golang noob myself).

matthiasbeyer commented 2 years ago

Awesome, I will see what I can do! Tomorrow though, TZ needs me in the bed :smile:

lyricnz commented 2 years ago

PR is merged now, so this should be workaround-able by binging quit to something else.