From 93c0056065946568583c633de63f5ad28c3275ce Mon Sep 17 00:00:00 2001 From: Peter Tripp Date: Thu, 27 Mar 2025 21:35:27 +0000 Subject: [PATCH] git: Display author not committer in git popover (#27628) Display Author name/email instead of Committer name/email in git popover. Closes https://github.com/zed-industries/zed/issues/27625 Prior to v0.173.x, Zed displayed Author name. Regression introduced in: - https://github.com/zed-industries/zed/pull/24593 Release Notes: - git: Switch to displaying Git author instead of Git committer in Git Blame popover. --- crates/editor/src/commit_tooltip.rs | 18 +++++++----------- crates/git_ui/src/git_panel.rs | 4 ++-- 2 files changed, 9 insertions(+), 13 deletions(-) diff --git a/crates/editor/src/commit_tooltip.rs b/crates/editor/src/commit_tooltip.rs index b9a3a444f7..432bfd2d80 100644 --- a/crates/editor/src/commit_tooltip.rs +++ b/crates/editor/src/commit_tooltip.rs @@ -20,8 +20,8 @@ use crate::hover_popover::hover_markdown_style; #[derive(Clone, Debug)] pub struct CommitDetails { pub sha: SharedString, - pub committer_name: SharedString, - pub committer_email: SharedString, + pub author_name: SharedString, + pub author_email: SharedString, pub commit_time: OffsetDateTime, pub message: Option, } @@ -133,16 +133,12 @@ impl CommitTooltip { CommitDetails { sha: blame.sha.to_string().into(), commit_time, - committer_name: blame - .committer_name + author_name: blame + .author .clone() .unwrap_or("".to_string()) .into(), - committer_email: blame - .committer_email - .clone() - .unwrap_or("".to_string()) - .into(), + author_email: blame.author_mail.clone().unwrap_or("".to_string()).into(), message: details, }, window, @@ -180,9 +176,9 @@ impl Render for CommitTooltip { fn render(&mut self, window: &mut Window, cx: &mut Context) -> impl IntoElement { let avatar = CommitAvatar::new(&self.commit).render(window, cx); - let author = self.commit.committer_name.clone(); + let author = self.commit.author_name.clone(); - let author_email = self.commit.committer_email.clone(); + let author_email = self.commit.author_email.clone(); let short_commit_id = self .commit diff --git a/crates/git_ui/src/git_panel.rs b/crates/git_ui/src/git_panel.rs index 4bffbb591c..2d96f0d82b 100644 --- a/crates/git_ui/src/git_panel.rs +++ b/crates/git_ui/src/git_panel.rs @@ -3951,8 +3951,8 @@ impl GitPanelMessageTooltip { let commit_details = editor::commit_tooltip::CommitDetails { sha: details.sha.clone(), - committer_name: details.committer_name.clone(), - committer_email: details.committer_email.clone(), + author_name: details.committer_name.clone(), + author_email: details.committer_email.clone(), commit_time: OffsetDateTime::from_unix_timestamp(details.commit_timestamp)?, message: Some(editor::commit_tooltip::ParsedCommitMessage { message: details.message.clone(),