guyfedwards / nom

RSS reader for the terminal
GNU General Public License v3.0
372 stars 25 forks source link

Move project towards using `bubbles/key` for keybinds. #64

Closed kanielrkirby closed 6 months ago

kanielrkirby commented 7 months ago

This would add some interoperability with features of bubbles/list and others that use this already, and makes setting up keybinds much more consistent. It shouldn't take too long, and would help address the PR for #61.

The main move would be creating three keybind keymaps, one for List View, one for Filter View (see #61), and one for the Viewport View, and transitioning the keymaps from raw strings to the keymap counterparts (e.g., case key.Matches(msg, ListKeyMap.Quit): instead of case "q", "ctrl+c", "etc.":)