[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [ELPA] New package: vertico
From: |
Tassilo Horn |
Subject: |
Re: [ELPA] New package: vertico |
Date: |
Sat, 10 Apr 2021 19:27:23 +0200 |
User-agent: |
mu4e 1.5.11; emacs 28.0.50 |
Stefan Monnier <monnier@iro.umontreal.ca> writes:
>>> Maybe a way to handle it "right" is for vertico to register an
>>> `after-change-function` so as to detect also when the buffer's
>>> content is modified outside of the normal commands (i.e. from a
>>> timer or process filter). It'd probably be a bit messy, tho: it
>>> wouldn't want to refresh the completion list right away, but there's
>>> no "obvious" later hook to use (like `post-command-hook`) so it
>>> would need to detect when there's not going to be a subsequent
>>> post-command-hook (i.e. when we're in a timer or similar) and then
>>> probably fire its own timer to update the completion list when the
>>> current processing is done. Or maybe it should use
>>> `after-change-function` to detect changes and
>>> `pre-redisplay-functions` to update the list of completion.
>>
>> Yes, there are other hooks that could be used. I like to keep it
>> simple as is and introduce nothing messy. I follow the design taken
>> by Icomplete which works well.
>
> FWIW, I'm not convinced either that the extra complexity of what
> I propose is worth the trouble.
FWIW, it's absolutely no big deal for me to call `vertico--exhibit'
after `minibuffer-complete' and it also doesn't seem to be costly as the
former already checks if the input changed before re-computing
completions. :-)
Bye,
Tassilo
- Re: [ELPA] New package: vertico, (continued)
- Re: [ELPA] New package: vertico, Tassilo Horn, 2021/04/10
- Re: [ELPA] New package: vertico, Daniel Mendler, 2021/04/10
- Re: [ELPA] New package: vertico, Tassilo Horn, 2021/04/10
- Re: [ELPA] New package: vertico, Daniel Mendler, 2021/04/10
- Re: [ELPA] New package: vertico, Stefan Monnier, 2021/04/10
- Re: [ELPA] New package: vertico, Daniel Mendler, 2021/04/10
- Re: [ELPA] New package: vertico, Stefan Monnier, 2021/04/10
- Re: [ELPA] New package: vertico,
Tassilo Horn <=
- Re: [ELPA] New package: vertico, Juri Linkov, 2021/04/10