From 62aba9e5ec8ccdb5a302b75fc6efb835c7eb3c70 Mon Sep 17 00:00:00 2001 From: Simek Date: Mon, 13 Oct 2025 20:02:02 +0200 Subject: [PATCH 1/2] git_ui: Layout/spacing tweaks for Blame tooltip --- crates/git_ui/src/blame_ui.rs | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/crates/git_ui/src/blame_ui.rs b/crates/git_ui/src/blame_ui.rs index b1dca288763c5c..5249536b76fb24 100644 --- a/crates/git_ui/src/blame_ui.rs +++ b/crates/git_ui/src/blame_ui.rs @@ -257,11 +257,11 @@ impl BlameRenderer for GitBlameRenderer { .child( v_flex() .w(gpui::rems(30.)) - .gap_4() .child( h_flex() - .pb_1p5() + .pb_1() .gap_x_2() + .items_center() .overflow_x_hidden() .flex_wrap() .children(avatar) @@ -280,6 +280,7 @@ impl BlameRenderer for GitBlameRenderer { div() .id("inline-blame-commit-message") .child(message) + .py_1() .max_h(message_max_height) .overflow_y_scroll() .track_scroll(&scroll_handle), @@ -289,7 +290,7 @@ impl BlameRenderer for GitBlameRenderer { .text_color(cx.theme().colors().text_muted) .w_full() .justify_between() - .pt_1p5() + .pt_1() .border_t_1() .border_color(cx.theme().colors().border_variant) .child(absolute_timestamp) From 2d9229cb2cb143e821ca07b76c3443e29646b6d5 Mon Sep 17 00:00:00 2001 From: Danilo Leal Date: Tue, 14 Oct 2025 09:54:22 -0300 Subject: [PATCH 2/2] Add some layout tweaks --- crates/git_ui/src/blame_ui.rs | 35 ++++++++++++++++------------------- 1 file changed, 16 insertions(+), 19 deletions(-) diff --git a/crates/git_ui/src/blame_ui.rs b/crates/git_ui/src/blame_ui.rs index 5249536b76fb24..469392ee1d8b28 100644 --- a/crates/git_ui/src/blame_ui.rs +++ b/crates/git_ui/src/blame_ui.rs @@ -17,7 +17,7 @@ use settings::Settings as _; use theme::ThemeSettings; use time::OffsetDateTime; use time_format::format_local_timestamp; -use ui::{ContextMenu, Divider, IconButtonShape, prelude::*, tooltip_container}; +use ui::{ContextMenu, Divider, prelude::*, tooltip_container}; use workspace::Workspace; const GIT_BLAME_MAX_AUTHOR_CHARS_DISPLAYED: usize = 20; @@ -61,16 +61,15 @@ impl BlameRenderer for GitBlameRenderer { .mr_2() .child( h_flex() + .id(("blame", ix)) .w_full() + .gap_2() .justify_between() .font_family(style.font().family) .line_height(style.line_height) - .id(("blame", ix)) .text_color(cx.theme().status().hint) - .gap_2() .child( h_flex() - .items_center() .gap_2() .child(div().text_color(sha_color).child(short_commit_id)) .children(avatar) @@ -250,8 +249,8 @@ impl BlameRenderer for GitBlameRenderer { }; Some( - tooltip_container(cx, |d, cx| { - d.occlude() + tooltip_container(cx, |this, cx| { + this.occlude() .on_mouse_move(|_, _, cx| cx.stop_propagation()) .on_mouse_down(MouseButton::Left, |_, _, cx| cx.stop_propagation()) .child( @@ -260,10 +259,11 @@ impl BlameRenderer for GitBlameRenderer { .child( h_flex() .pb_1() - .gap_x_2() - .items_center() + .gap_2() .overflow_x_hidden() .flex_wrap() + .border_b_1() + .border_color(cx.theme().colors().border_variant) .children(avatar) .child(author) .when(!author_email.is_empty(), |this| { @@ -272,18 +272,16 @@ impl BlameRenderer for GitBlameRenderer { .text_color(cx.theme().colors().text_muted) .child(author_email.to_owned()), ) - }) - .border_b_1() - .border_color(cx.theme().colors().border_variant), + }), ) .child( div() .id("inline-blame-commit-message") - .child(message) - .py_1() + .track_scroll(&scroll_handle) + .py_1p5() .max_h(message_max_height) .overflow_y_scroll() - .track_scroll(&scroll_handle), + .child(message), ) .child( h_flex() @@ -296,7 +294,7 @@ impl BlameRenderer for GitBlameRenderer { .child(absolute_timestamp) .child( h_flex() - .gap_1p5() + .gap_1() .when_some(pull_request, |this, pr| { this.child( Button::new( @@ -307,24 +305,24 @@ impl BlameRenderer for GitBlameRenderer { .icon(IconName::PullRequest) .icon_color(Color::Muted) .icon_position(IconPosition::Start) - .style(ButtonStyle::Subtle) + .icon_size(IconSize::Small) .on_click(move |_, _, cx| { cx.stop_propagation(); cx.open_url(pr.url.as_str()) }), ) + .child(Divider::vertical()) }) - .child(Divider::vertical()) .child( Button::new( "commit-sha-button", short_commit_id.clone(), ) - .style(ButtonStyle::Subtle) .color(Color::Muted) .icon(IconName::FileGit) .icon_color(Color::Muted) .icon_position(IconPosition::Start) + .icon_size(IconSize::Small) .on_click(move |_, window, cx| { CommitView::open( commit_summary.clone(), @@ -338,7 +336,6 @@ impl BlameRenderer for GitBlameRenderer { ) .child( IconButton::new("copy-sha-button", IconName::Copy) - .shape(IconButtonShape::Square) .icon_size(IconSize::Small) .icon_color(Color::Muted) .on_click(move |_, _, cx| {