notedeck

commit 7bae0945a7d0e1595c84afa45a2081ac876d98ce
parent 436c0413ec2911aae968cfaa479abec5e1c6896f
Author: William Casarin <jb55@jb55.com>
Date:   Fri, 30 Jan 2026 12:30:35 -0800

fix(dave): correct Ctrl+P/N keybinding action mapping

The previous commit had the keybinding actions swapped. Ctrl+N should
return FocusQueueNext (to jump to higher priority) and Ctrl+P should
return FocusQueuePrev (to drop to lower priority).

The comments correctly described the intent but the return values
were inverted.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Diffstat:
M
crates/notedeck_dave/src/ui/keybindings.rs
 | 
4
++--

1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/crates/notedeck_dave/src/ui/keybindings.rs b/crates/notedeck_dave/src/ui/keybindings.rs
@@ -81,12 +81,12 @@ pub fn check_keybindings(
 
     // Ctrl+N for higher priority (toward NeedsInput)
     if ctx.input(|i| i.modifiers.matches_exact(ctrl) && i.key_pressed(Key::N)) {
-        return Some(KeyAction::FocusQueuePrev);
+        return Some(KeyAction::FocusQueueNext);
     }
 
     // Ctrl+P for lower priority (toward Done)
     if ctx.input(|i| i.modifiers.matches_exact(ctrl) && i.key_pressed(Key::P)) {
-        return Some(KeyAction::FocusQueueNext);
+        return Some(KeyAction::FocusQueuePrev);
     }
 
     // Ctrl+T to spawn a new agent