From 7ca737fcf10f62b0e8239869b6bdb252e370d096 Mon Sep 17 00:00:00 2001 From: galuis116 <116897328+galuis116@users.noreply.github.com> Date: Mon, 20 Apr 2026 06:09:27 -0700 Subject: [PATCH] git_graph: Fix pluralization of "Changed Files" in commit details (#54284) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Self-Review Checklist: - [x] I've reviewed my own diff for quality, security, and reliability - [x] Unsafe blocks (if any) have justifying comments - [x] The content is consistent with the [UI/UX checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist) - [x] Tests cover the new/changed behavior - [x] Performance impact has been considered and is acceptable Closes #54283 Release Notes: - Fixed: Git graph commit detail header now uses “1 Changed File” when exactly one file changed, and “N Changed Files” otherwise, instead of always saying “Changed Files”. --------- Co-authored-by: Kunall Banerjee --- crates/git_graph/src/git_graph.rs | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/crates/git_graph/src/git_graph.rs b/crates/git_graph/src/git_graph.rs index 1d06ecb1c49fd05c2412c75c1cc6dd36eff84d9a..8f32887b03532573a8c57cd6d629fd8f3af7a76f 100644 --- a/crates/git_graph/src/git_graph.rs +++ b/crates/git_graph/src/git_graph.rs @@ -2100,9 +2100,17 @@ impl GitGraph { .w_full() .justify_between() .child( - Label::new(format!("{} Changed Files", changed_files_count)) - .size(LabelSize::Small) - .color(Color::Muted), + Label::new(format!( + "{} Changed {}", + changed_files_count, + if changed_files_count == 1 { + "File" + } else { + "Files" + } + )) + .size(LabelSize::Small) + .color(Color::Muted), ) .child(DiffStat::new( "commit-diff-stat",