From 7f8fd337b54d69578d4249fffffddfeb284fe308 Mon Sep 17 00:00:00 2001 From: Erik Kundt Date: Thu, 22 Jan 2026 13:25:40 +0100 Subject: [PATCH] bin: Streamline shortcuts --- bin/apps/inbox/list.rs | 28 ++++++++++++++++------------ bin/apps/issue/list.rs | 2 +- bin/apps/patch/list.rs | 30 +++++++++++++++++------------- 3 files changed, 34 insertions(+), 26 deletions(-) diff --git a/bin/apps/inbox/list.rs b/bin/apps/inbox/list.rs index 2f0c0bfd..037d27e1 100644 --- a/bin/apps/inbox/list.rs +++ b/bin/apps/inbox/list.rs @@ -106,7 +106,7 @@ impl Tui { } pub async fn run(&self) -> Result> { - let viewport = Viewport::Inline(20); + let viewport = Viewport::Fullscreen; let channel = Channel::default(); let state = App::try_from(&self.context)?; @@ -594,17 +594,21 @@ impl App { } pub fn show_browser_shortcuts(&self, frame: &mut Frame, ui: &mut Ui) { - ui.shortcuts( - frame, - &[ - ("enter", "show"), - ("r", "reload"), - ("c", "clear"), - ("/", "search"), - ("?", "help"), - ], - '∙', - Alignment::Left, + let shortcuts = vec![ + ("/", "search"), + ("enter", "show"), + ("c", "clear"), + ("r", "reload"), + ]; + let global_shortcuts = vec![("?", "help")]; + + ui.layout( + Layout::horizontal([Constraint::Fill(1), Constraint::Length(30)]), + None, + |ui| { + ui.shortcuts(frame, &shortcuts, '∙', Alignment::Left); + ui.shortcuts(frame, &global_shortcuts, '∙', Alignment::Right); + }, ); } diff --git a/bin/apps/issue/list.rs b/bin/apps/issue/list.rs index e833f287..70d8bc10 100644 --- a/bin/apps/issue/list.rs +++ b/bin/apps/issue/list.rs @@ -145,7 +145,7 @@ impl Tui { } pub async fn run(&self) -> Result> { - let viewport = Viewport::Inline(20); + let viewport = Viewport::Fullscreen; let channel = Channel::default(); let state = App::try_from(&self.context)?; diff --git a/bin/apps/patch/list.rs b/bin/apps/patch/list.rs index a4f5ffa3..fe1f1704 100644 --- a/bin/apps/patch/list.rs +++ b/bin/apps/patch/list.rs @@ -179,7 +179,7 @@ impl Tui { } pub async fn run(&self) -> Result> { - let viewport = Viewport::Inline(20); + let viewport = Viewport::Fullscreen; let channel = Channel::default(); let state = App::try_from(&self.context)?; @@ -721,18 +721,22 @@ impl App { } pub fn show_browser_shortcuts(&self, frame: &mut Frame, ui: &mut Ui) { - ui.shortcuts( - frame, - &[ - ("enter", "show"), - ("c", "checkout"), - ("d", "diff"), - ("/", "search"), - ("r", "reload"), - ("?", "help"), - ], - '∙', - Alignment::Left, + let shortcuts = vec![ + ("/", "search"), + ("enter", "show"), + ("c", "checkout"), + ("d", "diff"), + ("r", "reload"), + ]; + let global_shortcuts = vec![("?", "help")]; + + ui.layout( + Layout::horizontal([Constraint::Fill(1), Constraint::Length(30)]), + None, + |ui| { + ui.shortcuts(frame, &shortcuts, '∙', Alignment::Left); + ui.shortcuts(frame, &global_shortcuts, '∙', Alignment::Right); + }, ); }