Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

Support popup-style cycle-able diagnostics #10498

Closed
kas2020-commits opened this issue Apr 17, 2024 · 1 comment
Closed

Support popup-style cycle-able diagnostics #10498

kas2020-commits opened this issue Apr 17, 2024 · 1 comment
Labels
C-enhancement Category: Improvements

Comments

@kas2020-commits
Copy link
Contributor

First off, love both the Helix project and community, been using it as my main editor for over a year now!

A couple notes as primer before my ask:

so given that there's precedence now for a) adding a new way to see diagnostics and b) cycling through LSP results in a single popup window, I was hoping we could combine the 2 ideas and add the option to view diagnostics in a popup window where we can cycle through all the diagnostics at cursor position?

And before it's asked, yes, if the user choses this option (which would not be the default) then diagnostics won't get automatically shown on the screen. Instead, the user would manually have to use ]d, [d or a keymap that would show diagnostics under cursor (is that already implemented? AFAIK the answer's no but I'm not sure). I actually prefer this behavior - I'm not a huge fan of having a bunch of text show up then disappear when I'm quickly navigating my code.

But what do we think about this idea? Given that most IDEs, vscode and neovim all support this feature, I think it's UX lots of devs would be familiar with already. I'd be willing to drive this myself if such a feat would get approved.

@kas2020-commits kas2020-commits added the C-enhancement Category: Improvements label Apr 17, 2024
@pascalkuthe
Copy link
Member

I don't want to support additional ways to display diagnostic at some point its just bloat/too much config/too much maintaince requirement. I am very happy with the state inline diagnostics now

@helix-editor helix-editor locked and limited conversation to collaborators Apr 17, 2024
@pascalkuthe pascalkuthe converted this issue into discussion #10499 Apr 17, 2024

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
C-enhancement Category: Improvements
Projects
None yet
Development

No branches or pull requests

2 participants