Re: Select completions from the minibuffer

[Juri Linkov <juri@linkov.net>]

> I see that setting cursor-face-highlight-nonselected-window to t by
> default produces a highlight a bit annoying for some users...
>
> So I think that we may disable it and problem solved.
>
> For your selection from minibuffer I am not sure how you plan to enable
> it, but I suppose you will make a minor mode, so instead of setq-local
> on every call probably you will prefer to do something like:
>
> (defun minibuffer-select-completion-setup-function ()
>    (with-current-buffer standard-output
>       (when completions-highlight-face
>         (setq-local cursor-face-highlight-nonselected-window t))))
>
> (add-hook 'completion-setup-hook #'minibuffer-select-completion-setup-function)
>
> Is it fine for you?

Actually, such highlighting in non-selected completions window is needed
only for new commands that use the new macro with-minibuffer-completions-window,
because this macro navigates completions in the non-selected window,
so this works fine:

```
(defun minibuffer-previous-completion (&optional n)
  "Run `previous-completion' from the minibuffer in its completions window."
  (interactive "p")
  (with-minibuffer-completions-window
    (let ((completion-wrap-movement nil))
      (setq-local cursor-face-highlight-nonselected-window t)
      (previous-completion n))))
```