Skip to content

git: Replace file history view with git graph#50288

Merged
JosephTLyons merged 12 commits intomainfrom
repalce-file-view-backend
Apr 24, 2026
Merged

git: Replace file history view with git graph#50288
JosephTLyons merged 12 commits intomainfrom
repalce-file-view-backend

Conversation

@Anthony-Eid
Copy link
Copy Markdown
Contributor

@Anthony-Eid Anthony-Eid commented Feb 27, 2026

Summary

This PR replaces the git file history view with the git graph view that doesn't render the graph canvas. This has several advantages

  1. Benefits from the graphs performance and lazy loading
  2. Gets the graph's search for free
  3. Resizable columns
  4. The commit information panel
  5. Is persistent
  6. Cleans up a lot of code

The one con of this change is the graph doesn't have support remote/collab support yet, but that is a WIP and should be merged within a week.

Also, the git graph now propagates errors to the UI, which is the last thing on the graph's stable launch todo list!

Before you mark this PR as ready for review, make sure that you have:

  • Added a solid test coverage and/or screenshots from doing manual testing
  • Done a self-review taking into account security and performance aspects
  • Aligned any UI changes with the UI checklist

Release Notes:

  • Replaced the file history view with the git graph view, bringing improved performance, lazy loading, search, resizable columns, and the commit information panel.

Co-authored-by: dino <dinojoaocosta@gmail.com>
@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Feb 27, 2026
@zed-community-bot zed-community-bot Bot added the staff Pull requests authored by a current member of Zed staff label Feb 27, 2026
@Anthony-Eid Anthony-Eid marked this pull request as draft February 27, 2026 11:36
zed-zippy Bot and others added 2 commits February 27, 2026 11:54
* Use `--follow` instead of `follow`, duh
* Fix handling of failing `git log` command when fetching initial graph
  data

Co-authored-by: Anthony Eid <hello@anthonyeid.me>
@Anthony-Eid Anthony-Eid marked this pull request as ready for review April 7, 2026 10:20
Anthony-Eid added a commit that referenced this pull request Apr 7, 2026
Based on commit fba4980 that I worked
with Dino on in PR: #50288

Co-authored-by Dino <Dino@zed.dev>
Anthony-Eid added a commit that referenced this pull request Apr 7, 2026
Based on commit fba4980 that I worked
with Dino on in PR: #50288

Co-authored-by Dino \<Dino@zed.dev\>

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 #ISSUE

Release Notes:

- N/A or Added/Fixed/Improved ...
MasoudAlali pushed a commit to MasoudAlali/zed-ide that referenced this pull request Apr 7, 2026
…ries#53320)

Based on commit fba4980 that I worked
with Dino on in PR: zed-industries#50288

Co-authored-by Dino \<Dino@zed.dev\>

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 #ISSUE

Release Notes:

- N/A or Added/Fixed/Improved ...
github-actions Bot pushed a commit that referenced this pull request Apr 8, 2026
Based on commit fba4980 that I worked
with Dino on in PR: #50288

Co-authored-by Dino \<Dino@zed.dev\>

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 #ISSUE

Release Notes:

- N/A or Added/Fixed/Improved ...
Anthony-Eid added a commit that referenced this pull request Apr 8, 2026
…herry-pick to preview) (#53371)

Cherry-pick of #53320 to preview

----
Based on commit fba4980 that I worked
with Dino on in PR: #50288

Co-authored-by Dino \<Dino@zed.dev\>

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 #ISSUE

Release Notes:

- N/A or Added/Fixed/Improved ...

Co-authored-by: Anthony Eid <56899983+Anthony-Eid@users.noreply.github.com>
@JosephTLyons JosephTLyons self-requested a review April 24, 2026 02:51
@JosephTLyons JosephTLyons added this pull request to the merge queue Apr 24, 2026
Merged via the queue into main with commit 0194fe0 Apr 24, 2026
31 checks passed
@JosephTLyons JosephTLyons deleted the repalce-file-view-backend branch April 24, 2026 02:53
piper-of-dawn pushed a commit to piper-of-dawn/zed that referenced this pull request Apr 25, 2026
…ries#53320)

Based on commit fba4980 that I worked
with Dino on in PR: zed-industries#50288

Co-authored-by Dino \<Dino@zed.dev\>

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 #ISSUE

Release Notes:

- N/A or Added/Fixed/Improved ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement staff Pull requests authored by a current member of Zed staff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants