[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [ELPA] New package: vertico
From: |
Stefan Monnier |
Subject: |
Re: [ELPA] New package: vertico |
Date: |
Sat, 10 Apr 2021 09:34:35 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux) |
> If you bind `minibuffer-complete` to a keybinding in `vertico-map` it will
> automatically update the UI via the `post-command-hook`, which calls
> `vertico--exhibit`.
>
> I have checked your aggressive-completion package and there you are using
> a timer. I wonder why you are doing that (Performance issues?). Wouldn't it
> be better to use an `after-change-function` or a `post-command-hook`? If you
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.
Stefan
- Re: [ELPA] New package: vertico, (continued)
- Re: [ELPA] New package: vertico, Tassilo Horn, 2021/04/10
- Re: [ELPA] New package: vertico, Eli Zaretskii, 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, Tassilo Horn, 2021/04/10
- Re: [ELPA] New package: vertico, Daniel Mendler, 2021/04/10
- Re: [ELPA] New package: vertico,
Stefan Monnier <=
- 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, 2021/04/10
- Re: [ELPA] New package: vertico, Juri Linkov, 2021/04/10