From 9670cd3443b8e175b02c4c8740d025308940185b Mon Sep 17 00:00:00 2001 From: Daniel Edrisian Date: Mon, 29 Sep 2025 22:19:15 -0700 Subject: [PATCH 1/2] Move review against base branch to top --- codex-rs/tui/src/chatwidget.rs | 28 ++++++++++++++-------------- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/codex-rs/tui/src/chatwidget.rs b/codex-rs/tui/src/chatwidget.rs index 9a36addb0a..d855798e08 100644 --- a/codex-rs/tui/src/chatwidget.rs +++ b/codex-rs/tui/src/chatwidget.rs @@ -1778,6 +1778,20 @@ impl ChatWidget { pub(crate) fn open_review_popup(&mut self) { let mut items: Vec = Vec::new(); + items.push(SelectionItem { + name: "Review against a base branch (PR Style)".to_string(), + description: None, + is_current: false, + actions: vec![Box::new({ + let cwd = self.config.cwd.clone(); + move |tx| { + tx.send(AppEvent::OpenReviewBranchPicker(cwd.clone())); + } + })], + dismiss_on_select: false, + search_value: None, + }); + items.push(SelectionItem { name: "Review uncommitted changes".to_string(), description: None, @@ -1811,20 +1825,6 @@ impl ChatWidget { search_value: None, }); - items.push(SelectionItem { - name: "Review against a base branch".to_string(), - description: None, - is_current: false, - actions: vec![Box::new({ - let cwd = self.config.cwd.clone(); - move |tx| { - tx.send(AppEvent::OpenReviewBranchPicker(cwd.clone())); - } - })], - dismiss_on_select: false, - search_value: None, - }); - items.push(SelectionItem { name: "Custom review instructions".to_string(), description: None, From f443f11f106c78f9f10e84b6fcc0912c11ab8cff Mon Sep 17 00:00:00 2001 From: Daniel Edrisian Date: Mon, 29 Sep 2025 22:53:45 -0700 Subject: [PATCH 2/2] pr style desc --- codex-rs/tui/src/chatwidget.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/codex-rs/tui/src/chatwidget.rs b/codex-rs/tui/src/chatwidget.rs index d855798e08..981e0dfa94 100644 --- a/codex-rs/tui/src/chatwidget.rs +++ b/codex-rs/tui/src/chatwidget.rs @@ -1779,8 +1779,8 @@ impl ChatWidget { let mut items: Vec = Vec::new(); items.push(SelectionItem { - name: "Review against a base branch (PR Style)".to_string(), - description: None, + name: "Review against a base branch".to_string(), + description: Some("(PR Style)".into()), is_current: false, actions: vec![Box::new({ let cwd = self.config.cwd.clone();