-
Notifications
You must be signed in to change notification settings - Fork 164
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
Event for accepting a menu item #624
Comments
Would |
Yes, that was my idea. Or more specifically, I want my tab key to accept the selected menu selection, or if no menu was active apply history completion. The latter already works.
I'm not sure. I guess it's fine, but for me the enter key is really about executing the command. So if I browsed some menu suggestions but decide I don't want any of them I'd like to be able to execute the command, in theory. I'm not sure how often this scenario happens in practice though, but that would be ideal I think.
That also sounds decent to me, but I'm not sure if you could "unselect" the completion menu once you browsed through the completion options, in case you don't want any of the suggestions? Currently that doesn't seem possible with the completion menu. Another concern is that it doesn't quite match up with other tools I'm using, and not all tools provide the flexibility to behave like this. I think in principle both behaviors are fine, but depending on what other tools someone is using they may have a preference for one or the other. So in general we may want to support both options? |
I believe this is still valid. |
Related problem
I would like to change the keyboard binding for accepting a completion command. Currently this is "enter", but it would be nice if we could make this some other key. For executing the command I would like the key to remain "enter".
The reason I like to change this is to make my completion experience more uniform across different tools .
Describe the solution you'd like
Judging from the
keybindings list
command, there is not a MenuAccept event to be leveraged. Would it make sense to add such an event?Describe alternatives you've considered
There is already a command named "Enter", but this seems to cover both accepting a suggestion and executing the command.
I tried for instance
but this remaps the execute command key as well, which is not what I want.
Additional context and details
No response
The text was updated successfully, but these errors were encountered: