Re: Select completions from the minibuffer

[Ergus]

For a general case I am not sure this is correct or desired because the same window may change buffer.  And we want the highlight mode to be buffer local not window local right?... Even the new function in the predisplay hook is buffer local... I will find confusing such a mix...

Any way I am not the best lisper to have a strong opinion on this. But IMHO the code is actually pretty simple to deserve a more complex approach.

On March 19, 2022 8:03:34 PM GMT+01:00, Juri Linkov <juri@linkov.net> wrote:

 Eli just fixed the real issue in the display engine. So now your code
 does not need to force the update. BUT it still needs to set the

 (setq-local cursor-face-highlight-nonselected-window t)

 in completion-setup-function... I can do it for you unless you prefer to
 add your own function to completion-setup-hook.

Maybe better to set a new window parameter in the command that navigates
completions from the minibuffer.  Then such a window parameter could be
checked in redisplay--update-cursor-face-highlight.

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.