Skip to content

git_ui: Fix select first entry selects the wrong visual first entry when tree view is enabled#45108

Merged
cole-miller merged 1 commit intozed-industries:mainfrom
RemcoSmitsDev:git-ui-keyboard-navigation-follow-up-2
Dec 17, 2025
Merged

git_ui: Fix select first entry selects the wrong visual first entry when tree view is enabled#45108
cole-miller merged 1 commit intozed-industries:mainfrom
RemcoSmitsDev:git-ui-keyboard-navigation-follow-up-2

Conversation

@RemcoSmitsDev
Copy link
Contributor

@RemcoSmitsDev RemcoSmitsDev commented Dec 17, 2025

This PR fixes a bug where the select first didn't select the first visual entry when the first entry is a collapsed directory.

Follow-up: #45030

Before:

Screen.Recording.2025-12-17.at.14.08.19.mov

After:

Screen.Recording.2025-12-17.at.14.09.08.mov

Release Notes:

  • git_ui: Fix select first entry selects the wrong visual first entry when tree view is enabled

mode is enabled.

Before this commit we would select a entry that is visually not the
first, but logical wise it is. This happends if the first entry is a
directory with files that are collapsed. We would select entry n of
entries in the directory that is collapsed after it.

Follow-up: zed-industries#45030
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Dec 17, 2025
@github-actions github-actions bot added the community champion Issues filed by our amazing community champions! 🫶 label Dec 17, 2025
@github-project-automation github-project-automation bot moved this to Community Champion PRs in Quality Week – December 2025 Dec 17, 2025
@cole-miller
Copy link
Member

Thanks!

@cole-miller cole-miller enabled auto-merge (squash) December 17, 2025 15:22
@cole-miller cole-miller merged commit 5b8e4e5 into zed-industries:main Dec 17, 2025
25 checks passed
@github-project-automation github-project-automation bot moved this from Community Champion PRs to Done in Quality Week – December 2025 Dec 17, 2025
HactarCE pushed a commit that referenced this pull request Dec 17, 2025
…hen tree view is enabled (#45108)

This PR fixes a bug where the select first didn't select the first
visual entry when the first entry is a collapsed directory.

Follow-up: #45030

**Before**:


https://github.com/user-attachments/assets/5e5865cc-ec0f-471d-a81b-9521fb70df41

**After**:


https://github.com/user-attachments/assets/05562572-e43f-4d1e-9638-80e4dccc0998

Release Notes:

- git_ui: Fix select first entry selects the wrong visual first entry
when tree view is enabled
rtfeldman pushed a commit that referenced this pull request Jan 5, 2026
…hen tree view is enabled (#45108)

This PR fixes a bug where the select first didn't select the first
visual entry when the first entry is a collapsed directory.

Follow-up: #45030

**Before**:


https://github.com/user-attachments/assets/5e5865cc-ec0f-471d-a81b-9521fb70df41

**After**:


https://github.com/user-attachments/assets/05562572-e43f-4d1e-9638-80e4dccc0998

Release Notes:

- git_ui: Fix select first entry selects the wrong visual first entry
when tree view is enabled
LivioGama pushed a commit to LivioGama/zed that referenced this pull request Jan 20, 2026
…hen tree view is enabled (zed-industries#45108)

This PR fixes a bug where the select first didn't select the first
visual entry when the first entry is a collapsed directory.

Follow-up: zed-industries#45030

**Before**:


https://github.com/user-attachments/assets/5e5865cc-ec0f-471d-a81b-9521fb70df41

**After**:


https://github.com/user-attachments/assets/05562572-e43f-4d1e-9638-80e4dccc0998

Release Notes:

- git_ui: Fix select first entry selects the wrong visual first entry
when tree view is enabled
LivioGama pushed a commit to LivioGama/zed that referenced this pull request Jan 20, 2026
…hen tree view is enabled (zed-industries#45108)

This PR fixes a bug where the select first didn't select the first
visual entry when the first entry is a collapsed directory.

Follow-up: zed-industries#45030

**Before**:


https://github.com/user-attachments/assets/5e5865cc-ec0f-471d-a81b-9521fb70df41

**After**:


https://github.com/user-attachments/assets/05562572-e43f-4d1e-9638-80e4dccc0998

Release Notes:

- git_ui: Fix select first entry selects the wrong visual first entry
when tree view is enabled
LivioGama pushed a commit to LivioGama/zed that referenced this pull request Feb 15, 2026
…hen tree view is enabled (zed-industries#45108)

This PR fixes a bug where the select first didn't select the first
visual entry when the first entry is a collapsed directory.

Follow-up: zed-industries#45030

**Before**:


https://github.com/user-attachments/assets/5e5865cc-ec0f-471d-a81b-9521fb70df41

**After**:


https://github.com/user-attachments/assets/05562572-e43f-4d1e-9638-80e4dccc0998

Release Notes:

- git_ui: Fix select first entry selects the wrong visual first entry
when tree view is enabled
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 community champion Issues filed by our amazing community champions! 🫶

Projects

Development

Successfully merging this pull request may close these issues.

2 participants