Skip to content

Conversation

@RemcoSmitsDev
Copy link
Contributor

This PR adds support for collapsing/expanding Git entries with your keyboard like you can inside the project panel and variable list.

I noticed there is a bug that selecting the next entry when you are on the directory level will select a non-visible entry. Will fix that in another PR, as it is not related to this feature implementation.

Result:

Screen.Recording.2025-12-16.at.16.46.08.mov

Release Notes:

  • Git panel: Add support for collapsing/expanding entries with your keyboard.
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Dec 16, 2025
@github-actions github-actions bot added the community champion Issues filed by our amazing community champions! 🫶 label Dec 16, 2025
@github-project-automation github-project-automation bot moved this to Community Champion PRs in Quality Week – December 2025 Dec 16, 2025
@cole-miller
Copy link
Member

Thanks!

@cole-miller cole-miller enabled auto-merge (squash) December 16, 2025 16:29
auto-merge was automatically disabled December 16, 2025 17:13

Head branch was pushed to by a user without write access

@RemcoSmitsDev
Copy link
Contributor Author

It should be good now!

@cole-miller cole-miller enabled auto-merge (squash) December 16, 2025 17:33
@cole-miller cole-miller merged commit 3f11cbd into zed-industries:main Dec 16, 2025
23 checks passed
@github-project-automation github-project-automation bot moved this from Community Champion PRs to Done in Quality Week – December 2025 Dec 16, 2025
@RemcoSmitsDev RemcoSmitsDev deleted the git-ui-keyboard-navigation branch December 16, 2025 17:52
RemcoSmitsDev added a commit to RemcoSmitsDev/zed that referenced this pull request Dec 16, 2025
Before this commit we would select a non-visible entry when a directory
is collapsed. No we correctly select the visible entry that is visually
the previous/next entry.

Follow-up: zed-industries#45002
RemcoSmitsDev added a commit to RemcoSmitsDev/zed that referenced this pull request Dec 16, 2025
Before this commit we would select a non-visible entry when a directory
is collapsed. No we correctly select the visible entry that is visually
the previous/next entry.

Follow-up: zed-industries#45002
cole-miller pushed a commit that referenced this pull request Dec 16, 2025
… tree view is enabled (#45030)

Before this commit, we would select a non-visible entry when a directory
is collapsed. Now we correctly select the visible entry that is visually
the previous/next entry in the list.

**Note**: I removed the `cx.notify()` call as it's already part of the
`self.scroll_to_selected_entry(cx)` call. So we don't notify twice :).

Follow-up: #45002

**Before**


https://github.com/user-attachments/assets/da0b8084-0081-4d98-ad8a-c11c3b95a1b7

**After**


https://github.com/user-attachments/assets/8a16afb0-fdde-4317-b419-13143d5d608e

Release Notes:

- git_ui: Fix select next/previous entry selects non-visible entry when
tree view is enabled
HactarCE pushed a commit that referenced this pull request Dec 17, 2025
… tree view is enabled (#45030)

Before this commit, we would select a non-visible entry when a directory
is collapsed. Now we correctly select the visible entry that is visually
the previous/next entry in the list.

**Note**: I removed the `cx.notify()` call as it's already part of the
`self.scroll_to_selected_entry(cx)` call. So we don't notify twice :).

Follow-up: #45002

**Before**


https://github.com/user-attachments/assets/da0b8084-0081-4d98-ad8a-c11c3b95a1b7

**After**


https://github.com/user-attachments/assets/8a16afb0-fdde-4317-b419-13143d5d608e

Release Notes:

- git_ui: Fix select next/previous entry selects non-visible entry when
tree view is enabled
LivioGama pushed a commit to LivioGama/zed that referenced this pull request Jan 20, 2026
…rd (zed-industries#45002)

This PR adds support for collapsing/expanding Git entries with your
keyboard like you can inside the project panel and variable list.

I noticed there is a bug that selecting the next entry when you are on
the directory level will select a non-visible entry. Will fix that in
another PR, as it is not related to this feature implementation.

**Result**:


https://github.com/user-attachments/assets/912cc146-1e1c-485f-9b60-5ddc0a124696

Release Notes:

- Git panel: Add support for collapsing/expanding entries with your
keyboard.
LivioGama pushed a commit to LivioGama/zed that referenced this pull request Jan 20, 2026
… tree view is enabled (zed-industries#45030)

Before this commit, we would select a non-visible entry when a directory
is collapsed. Now we correctly select the visible entry that is visually
the previous/next entry in the list.

**Note**: I removed the `cx.notify()` call as it's already part of the
`self.scroll_to_selected_entry(cx)` call. So we don't notify twice :).

Follow-up: zed-industries#45002

**Before**


https://github.com/user-attachments/assets/da0b8084-0081-4d98-ad8a-c11c3b95a1b7

**After**


https://github.com/user-attachments/assets/8a16afb0-fdde-4317-b419-13143d5d608e

Release Notes:

- git_ui: Fix select next/previous entry selects non-visible entry when
tree view is enabled
LivioGama pushed a commit to LivioGama/zed that referenced this pull request Jan 20, 2026
…rd (zed-industries#45002)

This PR adds support for collapsing/expanding Git entries with your
keyboard like you can inside the project panel and variable list.

I noticed there is a bug that selecting the next entry when you are on
the directory level will select a non-visible entry. Will fix that in
another PR, as it is not related to this feature implementation.

**Result**:


https://github.com/user-attachments/assets/912cc146-1e1c-485f-9b60-5ddc0a124696

Release Notes:

- Git panel: Add support for collapsing/expanding entries with your
keyboard.
LivioGama pushed a commit to LivioGama/zed that referenced this pull request Jan 20, 2026
… tree view is enabled (zed-industries#45030)

Before this commit, we would select a non-visible entry when a directory
is collapsed. Now we correctly select the visible entry that is visually
the previous/next entry in the list.

**Note**: I removed the `cx.notify()` call as it's already part of the
`self.scroll_to_selected_entry(cx)` call. So we don't notify twice :).

Follow-up: zed-industries#45002

**Before**


https://github.com/user-attachments/assets/da0b8084-0081-4d98-ad8a-c11c3b95a1b7

**After**


https://github.com/user-attachments/assets/8a16afb0-fdde-4317-b419-13143d5d608e

Release Notes:

- git_ui: Fix select next/previous entry selects non-visible 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! 🫶

2 participants