Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make git panel entries clickable #22329

Merged
merged 9 commits into from
Dec 21, 2024
Merged

Conversation

SomeoneToIgnore
Copy link
Contributor

Makes a first pass over git panel UI, making it more interactive.

image

  • every item can be selected, the selection is shown in the panel
  • every item can be clicked, which changes the selection and creates/focuses the editor with a project changes multi buffer
  • the editor is scrolled so that the clicked item is in the center
  • it's possible to nagivate up and down the panel, selecting next/previous items in it, triggering the editor scroll

Known issues:

  • entries are updated oddly sometimes (should become better after DiffMap improvements land?)
  • only unstaged diffs are shown currently (entry status storage should help with this)
  • no deleted files are displayed (the underlying work is done by others now)
  • added files have no diff hunks shown (DiffMap will have it?)
  • performance story has not improved (again, DiffMap and status storage should help with this)

Release Notes:

  • N/A

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Dec 21, 2024
@SomeoneToIgnore SomeoneToIgnore added this pull request to the merge queue Dec 21, 2024
Merged via the queue into main with commit 831930a Dec 21, 2024
13 checks passed
@SomeoneToIgnore SomeoneToIgnore deleted the kb/git-panel-multi-buffer branch December 21, 2024 14:21
@Angelk90

This comment has been minimized.

CharlesChen0823 pushed a commit to CharlesChen0823/zed that referenced this pull request Dec 24, 2024
Makes a first pass over git panel UI, making it more interactive.


![image](https://github.com/user-attachments/assets/4d43b086-4ef2-4913-9783-2b9467d99c9a)


* every item can be selected, the selection is shown in the panel
* every item can be clicked, which changes the selection and
creates/focuses the editor with a project changes multi buffer
* the editor is scrolled so that the clicked item is in the center
* it's possible to nagivate up and down the panel, selecting
next/previous items in it, triggering the editor scroll

Known issues:

* entries are updated oddly sometimes (should become better after
DiffMap improvements land?)
* only unstaged diffs are shown currently (entry status storage should
help with this)
* no deleted files are displayed (the underlying work is done by others
now)
* added files have no diff hunks shown (DiffMap will have it?)
* performance story has not improved (again, DiffMap and status storage
should help with this)

Release Notes:

- N/A
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants