commit caa473351a0628e8b2eb5f545303df624c9edd25
parent cf61c74642c9a7f359fa484450db50e3cb75e57a
Author: William Casarin <jb55@jb55.com>
Date: Tue, 27 Jan 2026 10:47:22 -0800
dave: move approve/deny buttons to bottom right of diff view
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Diffstat:
1 file changed, 10 insertions(+), 5 deletions(-)
diff --git a/crates/notedeck_dave/src/ui/dave.rs b/crates/notedeck_dave/src/ui/dave.rs
@@ -338,14 +338,19 @@ impl<'a> DaveUi<'a> {
.corner_radius(corner_radius)
.stroke(egui::Stroke::new(1.0, ui.visuals().warn_fg_color))
.show(ui, |ui| {
- // Header with file path and buttons
- ui.horizontal(|ui| {
- diff::file_path_header(&file_update, ui);
- self.permission_buttons(request, ui, &mut action);
- });
+ // Header with file path
+ diff::file_path_header(&file_update, ui);
// Diff view
diff::file_update_ui(&file_update, ui);
+
+ // Approve/deny buttons at the bottom right
+ ui.with_layout(
+ egui::Layout::right_to_left(egui::Align::Center),
+ |ui| {
+ self.permission_buttons(request, ui, &mut action);
+ },
+ );
});
} else {
// Parse tool input for display (existing logic)