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

Less jumping around due to folds when opening comment threads during review #536

Open
GuillaumeLagrange opened this issue Apr 17, 2024 · 0 comments

Comments

@GuillaumeLagrange
Copy link
Contributor

Issue Description

Type: feature request

Describe what happened (or what feature you want)

When reviewing a PR, in the file diff view, I find it a bit hard to follow all the jumping around that folds are causing when moving cursor over a line which has a comment thread.
The root cause for this is that the other window of the split (most often the left window in my case) is replaced by the comment thread view, so the diff view stops showing its colors + the folds are automatically open, causing a lot of visual noise IMO.

There is also the annoyance of the thread window going away if I move around the file, when sometimes I am writing/reading a comment and want to check something 20 lines above.

Describe what you expected to happen

  1. First, I would like an option to only trigger comment thread display through a dedicated command, to prevent jumps when just moving around the file. Inside the comment thread buffer, q in normal mode would close the buffer and go back to diff view (this could/should be bindable to let users chose their keybinds)

  2. To go even further, we could maybe make use of nvim's floating windows ? Something like lsp hover windows where hovering over a line with comments, to avoid covering up the other part of the diff. I am less sure about this.

I have a more or less working version of step 1 that disables the autocommand that shows threads and adds support for manually opening/closing the thread buffer. I could open a PR, I am opening this issue to gauge interest.

Cheers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant