[BUG] Inconsistency with RadioBrowser Config's vertical navigation keybindings (j/k)

[bash-ful]

Describe the bug
The navigation keys 'j' and 'k' for the RadioBrowser Config window is inconsistent with the other windows' functionality of the keys. On other windows, j moves the cursor up, k moves the cursor down. It is reversed on said config window.

To Reproduce
Steps to reproduce the behavior:

1. Open RadioBrowser (O).
2. Open RadioBrowserConfig (c)
3. Press j and/or k

Expected behavior
To stay consistent with other windows' functionality of each key, namely:
j - moves the cursor down
k - moves the cursor up

System (please complete the following information):

• OS: Arch Linux
• Python version: 3.13.7 (main, Aug 15 2025, 12:34:02) [GCC 15.2.1 20250813]
• pipx version (if applicable):
• pipx installation method: Fully isolated
• PyRadio Version: 0.9.3.11.19

Additional context
I would like to note that the PyRadio Main Help (?) states:

You can use the following keys to navigate: j (Up), k (Down) ...

also, from Playlist Help (?) may imply that j is up and k is down:

Up, j, PgUp,
Down, k, PgDown

even though on both of those windows, the functionality is reversed.

and finally, from RadioBrowser Config Help (?):

j, Up / k, Down

if the Help pages are to be believed, RadioBrowser Config's j and k functionality is actually the correct one, despite it being the odd one out from the other windows, not to mention that it is in fact reversed compared to vi(m)'s keybindings.

[bash-ful]

I am willing to submit a PR, though i do need to know what the intended functionality is

[s-n-g]

Hi @bash-ful

Sorry it took me so long to reply

About the bug, that was me being sloppy...

It's such an easy fix, though (already fixed in devel)

diff --git a/pyradio/browser.py b/pyradio/browser.py
--- a/pyradio/browser.py
+++ b/pyradio/browser.py
@@ -2436,12 +2436,12 @@ class RadioBrowserConfigWindow():
 
         elif char in (curses.KEY_UP, kbkey['j']) or \
                 check_localized(char, (kbkey['j'], )):
-            self._focus_previous()
+            self._focus_next()
             self.calculate_dirty()
 
         elif char in (curses.KEY_DOWN, kbkey['k']) or \
                 check_localized(char, (kbkey['k'], )):
-            self._focus_next()
+            self._focus_previous()
             self.calculate_dirty()
 
         else:

This is a patch that will fix it

If you want to apply it, save it in ~/.local/pipx/venvs/pyradio/lib/python3.13/site-packages (call it 309.py, for example).
Open a terminal and

cd ~/.local/pipx/venvs/pyradio/lib/python3.13/site-packages
patch -p1 < 309.patch

And you are done

I will leave this issue open until a new release is out

[s-n-g]

released: https://github.com/coderholic/pyradio/releases/tag/0.9.3.11.20