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

Signature help gets stuck #3112

Closed
theli-ua opened this issue Jul 19, 2022 · 4 comments · Fixed by #4456
Closed

Signature help gets stuck #3112

theli-ua opened this issue Jul 19, 2022 · 4 comments · Fixed by #4456
Assignees
Labels
A-helix-term Area: Helix term improvements C-bug Category: This is a bug

Comments

@theli-ua
Copy link
Contributor

Summary

Occasionally signature help gets stuck without (at least I couldn't find it) ability to to hide it.
I am yet to come up with minimal reproduction example but once I find it I'll update this.

(this all not to say that signature help in general makes experience incredibly frustrating inside things as iterator::map or , say, tokio::spawn)

Reproduction Steps

pending

Helix log

~/.cache/helix/helix.log
I am unable to share full log and could not find anything relevant there

Platform

Linux

Terminal Emulator

kitty

Helix Version

helix 22.05 (791bf7e)

@theli-ua theli-ua added the C-bug Category: This is a bug label Jul 19, 2022
@kirawi
Copy link
Member

kirawi commented Jul 19, 2022

Does esc or ctrl-c not work to close it?

@n0s4
Copy link
Contributor

n0s4 commented Jul 19, 2022

Does esc or ctrl-c not work to close it?

For me C-c works to close it but not esc.

I also managed to reproduce it by creating a macro that just enters and then exits insert mode, then running it with the cursor on a function parameter.

@kirawi kirawi added the A-helix-term Area: Helix term improvements label Jul 19, 2022
@theli-ua
Copy link
Contributor Author

C-c works indeed

@the-mikedavis
Copy link
Member

When I see this get stuck, I think it's because the signature help reply arrives when we're in normal mode. I see this getting stuck much more often with a slow language server (erlang_ls) rather than a fast language server (rust-analyzer). When it arrives in insert mode and we switch to normal mode, the popup is closed by this block:

cxt.jobs.callback(async {
let call: job::Callback = Box::new(|_editor, compositor| {
compositor.remove(SignatureHelp::ID);
});
Ok(call)
});

I think it would be a good improvement to discard the signature help reply when the editor is in normal mode.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-helix-term Area: Helix term improvements C-bug Category: This is a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants