diff options
| author | 2025-10-14 11:10:08 +0200 | |
|---|---|---|
| committer | 2025-10-14 11:10:08 +0200 | |
| commit | 71a58415b1fe9c6b887972b636f43afdf08d3b3e (patch) | |
| tree | bd0a64a4ccc9c684fff53a7ed8bb2bd91c90eadb /p/scripts/main.js | |
| parent | 5eba322cbd24191e05304df08c80af846977d99b (diff) | |
Fix navigating between read feeds using shortcut shift+j/k (#8057)
Before, the shortcuts marked in red and blue behaved the same way (with shift modifier)
Specifically <kbd>Shift + J</kbd> had the same behavior as <kbd>Shift + K</kbd> which means it only jumped to unread and not read/unread.
<img width="719" height="396" alt="image" src="https://github.com/user-attachments/assets/8ebd1efc-c186-4dcf-9b54-b9acbf3bbbe5" />
Now the shift modifier shortcuts match the behavior of the alt and no modifier shortcuts.
<kbd>Shift + K</kbd> was corrected too, but there is no alternative for it yet, since I don't know which default key should be used for it - <kbd>l</kbd> is already taken by *My labels* shortcut
Diffstat (limited to 'p/scripts/main.js')
| -rw-r--r-- | p/scripts/main.js | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/p/scripts/main.js b/p/scripts/main.js index 8f85f7333..2c0912a7e 100644 --- a/p/scripts/main.js +++ b/p/scripts/main.js @@ -569,7 +569,7 @@ function next_unread_entry(skipping) { toggleContent(new_active, old_active, skipping); } -function prev_feed() { +function prev_feed(jump_to_unread) { let found = false; let adjacent = null; const feeds = document.querySelectorAll('#aside_feed .feed'); @@ -585,7 +585,7 @@ function prev_feed() { if (getComputedStyle(feed).display === 'none') { continue; } - if (feed.dataset.unread != 0) { + if (jump_to_unread && feed.dataset.unread != 0) { delayedClick(feed.querySelector('a.item-title')); return; } else if (adjacent === null) { @@ -599,7 +599,7 @@ function prev_feed() { } } -function next_feed() { +function next_feed(jump_to_unread) { let found = false; let adjacent = null; const feeds = document.querySelectorAll('#aside_feed .feed'); @@ -615,7 +615,7 @@ function next_feed() { if (getComputedStyle(feed).display === 'none') { continue; } - if (feed.dataset.unread != 0) { + if (jump_to_unread && feed.dataset.unread != 0) { delayedClick(feed.querySelector('a.item-title')); return; } else if (adjacent === null) { @@ -1090,7 +1090,7 @@ function init_shortcuts() { if (ev.altKey) { next_category(); } else if (ev.shiftKey) { - next_feed(); + next_feed(false); } else { next_entry(false); } @@ -1101,7 +1101,7 @@ function init_shortcuts() { if (ev.altKey) { next_unread_category(); } else if (ev.shiftKey) { - next_feed(); + next_feed(true); } else { next_unread_entry(false); } @@ -1112,7 +1112,7 @@ function init_shortcuts() { if (ev.altKey) { prev_category(); } else if (ev.shiftKey) { - prev_feed(); + prev_feed(false); } else { prev_entry(false); } |
