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

LSP signature helper should not appear when inside a lambda #3385

Closed
dead10ck opened this issue Aug 10, 2022 · 2 comments
Closed

LSP signature helper should not appear when inside a lambda #3385

dead10ck opened this issue Aug 10, 2022 · 2 comments
Labels
A-helix-term Area: Helix term improvements A-language-server Area: Language server client upstream

Comments

@dead10ck
Copy link
Member

dead10ck commented Aug 10, 2022

The new LSP signature helper pops up when you're inside the body of a lambda that's being passed to a function, which is a little annoying.

We could probably fix it by only making the pop-up visible when the node under the cursor is a direct descendent of the function arguments.

@dead10ck dead10ck added the C-bug Category: This is a bug label Aug 10, 2022
@the-mikedavis the-mikedavis added A-language-server Area: Language server client A-helix-term Area: Helix term improvements labels Aug 10, 2022
@the-mikedavis
Copy link
Member

the-mikedavis commented Aug 10, 2022

I was thinking the same about it being noisy when writing funs in Erlang erlang-ls/erlang_ls#1344. It might be good to propose this upstream in rust-analyzer as well.

The direct descendant idea I think is interesting though. I would definitely be interested in trying that out if it existed. I suspect it may have some bugs though:

  • it could be too selective
    • for example, typing tuples of variables in a function call in rust I would like to continue seeing the signature help
    • in these cases it might be better to have a manual keybind to summon signature-help like auto-complete's C-x
  • I'm not sure how well it would work for temporarily errored syntax trees
    • if you're typing a comma to separate params and the grammar is very unforgiving, would it complicate the direct descendant check?

@dead10ck
Copy link
Member Author

Hmm, yeah, you're right. It seems like where it is desirable and where it is not is very language specific, so a solution might not be simple. A manual trigger would be helpful, because right now it's either all or nothing.

@pascalkuthe pascalkuthe added upstream and removed C-bug Category: This is a bug labels Apr 4, 2024
@helix-editor helix-editor locked and limited conversation to collaborators Apr 14, 2024
@pascalkuthe pascalkuthe converted this issue into discussion #10416 Apr 14, 2024

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
A-helix-term Area: Helix term improvements A-language-server Area: Language server client upstream
Projects
None yet
Development

No branches or pull requests

3 participants