-
-
Notifications
You must be signed in to change notification settings - Fork 65
Completion update #788
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
base: master
Are you sure you want to change the base?
Completion update #788
Conversation
This replaces consult-preview-at-point-mode
Introduce embark--with-completion-list-buffer which switches to an active completions buffer if there is one tied to the current minibuffer.
The completion list candidate takes precendence if there is one. Only if no candidate has been selected, take the top minibuffer candidate.
|
For testing I recommend the settings from https://github.com/minad/consult?tab=readme-ov-file#default-completion-ui-with-auto-update. These settings make the default completion UI more comfortable to use. |
|
Oh, is this something I need to test on the current master? I'm using the released Emacs 30.1. Since I have now lost most of my fear of compiling Emacs from source I guess I could test this. |
|
Yes, you need Emacs master. I had forgotten that you've just moved to Emacs 30 a while ago. :/ But if you can test on Emacs 30 it would also be good, I guess. My main concern is the reordering of the target finder. I am not sure if this has unexpected implications. |
708c4bf to
f21dedb
Compare
Fixes #787. Please test carefully. You need
minibuffer-visible-completionsset totin order to see the effect. This settings allows you to "remote" control the Completions buffer from the minibuffer. You can navigate with up and down keys from the minibuffer.