all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Ergus <spacibba@aol.com>
To: Juri Linkov <juri@linkov.net>
Cc: Lars Ingebrigtsen <larsi@gnus.org>, emacs-devel@gnu.org
Subject: Re: Select completions from the minibuffer
Date: Thu, 17 Mar 2022 23:44:33 +0100	[thread overview]
Message-ID: <20220317224433.oiocm6da6tzmrq6e@Ergus> (raw)
In-Reply-To: <865yoc47d2.fsf@mail.linkov.net>

Looking around it seems like with your function the *Completions* window
is not passed to redisplay--pre-redisplay-functions, but only the
minibuffer. So basically the window is not updated and the overlay not
moved. Probably there is some optimization that is affecting us... Eli?



On Thu, Mar 17, 2022 at 10:56:01PM +0200, Juri Linkov wrote:
>>> But cursor-face is not highlighted in *Completions* when the current buffer
>>> is the minibuffer.  Such highlighting is necessary when completions
>>> are navigated from the minibuffer.
>
>> Please check the change and try to set
>> cursor-face-highlight-nonselected-window unconditionally in
>> *Completions* to t when using your mode.
>
>Now it keeps highlighting in *Completions* after switching to the minibuffer,
>but still doesn't move the highlighting in *Completions* when
>completions are navigated from the minibuffer with
>
>  (defmacro with-minibuffer-completions-window (&rest body)
>    "Execute the forms in BODY from the minibuffer in its completions window.
>  When used in a minibuffer window, select the window with completions,
>  and execute the forms."
>    (declare (indent 0) (debug t))
>    `(let ((window (or (get-buffer-window "*Completions*" 0)
>                       ;; Make sure we have a completions window.
>                       (progn (minibuffer-completion-help)
>                              (get-buffer-window "*Completions*" 0)))))
>       (when window
>         (with-selected-window window
>           ,@body))))
>
>  (defun minibuffer-next-completion (&optional n)
>    "Run `next-completion' from the minibuffer in its completions window."
>    (interactive "p")
>    (with-minibuffer-completions-window
>      (next-completion n)))
>
>> If you could improve the initialization in order to use
>> cursor-in-non-selected-windows unless the user sets another value
>> manually it may be nicer (because in the general scenario the user could
>> set cursor-in-non-selected-windows latter so this variable may have an
>> outdated value) otherwise consider if this may be simply t or nil by
>> default. But at the moment it works fine for the current use cases.
>
>I think it's unrelated to cursor-in-non-selected-windows
>because the cursor in non-selected windows is displayed as
>a hollow box, but cursor-face stays the same.
>So you can set cursor-face-highlight-nonselected-window
>to nil by default.  Then commands could let-bind it like
>
>  (defun minibuffer-next-completion (&optional n)
>    "Run `next-completion' from the minibuffer in its completions window."
>    (interactive "p")
>    (let ((cursor-face-highlight-nonselected-window t))
>      (with-minibuffer-completions-window
>        (next-completion n))))
>
>(but this currently doesn't work for the same reason as above)
>



  parent reply	other threads:[~2022-03-17 22:44 UTC|newest]

Thread overview: 92+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-03-10 18:58 Select completions from the minibuffer Juri Linkov
2022-03-10 19:32 ` Lars Ingebrigtsen
2022-03-10 19:51   ` Juri Linkov
2022-03-10 20:32     ` Lars Ingebrigtsen
2022-03-11  8:58       ` Juri Linkov
2022-03-12 17:08         ` Lars Ingebrigtsen
2022-03-12 18:55           ` Juri Linkov
2022-03-13 14:05             ` Lars Ingebrigtsen
2022-03-13 18:05               ` Juri Linkov
2022-03-14  9:13                 ` Lars Ingebrigtsen
2022-03-21 19:28                   ` Juri Linkov
2022-03-21 19:30                     ` Lars Ingebrigtsen
2022-03-22  8:24                       ` Juri Linkov
2022-03-22 13:38                         ` Eli Zaretskii
2022-03-22 13:44                           ` Eric S Fraga
2022-03-22 14:18                           ` Ergus
2022-03-22 14:49                             ` Eli Zaretskii
2022-03-22 15:21                               ` Ergus
2022-03-22 18:38                                 ` Juri Linkov
2022-03-22 19:07                                   ` Ergus
2022-03-22 19:24                                   ` Ergus
2022-03-23  8:47                                     ` Juri Linkov
2022-03-23 12:07                                       ` Ergus
2022-03-23 18:28                                         ` Juri Linkov
2022-03-24  9:07                                           ` Ergus
2022-03-22 15:46                               ` Ergus
2022-03-22 16:59                                 ` Eli Zaretskii
2022-03-22 17:10                                   ` Ergus
2022-03-22 17:58                               ` Eli Zaretskii
2022-03-22 17:56                             ` Juri Linkov
2022-03-22 17:23                         ` Ergus
2022-03-22 17:52                           ` Juri Linkov
2022-03-22 18:31                           ` Juri Linkov
2022-03-14  8:41               ` Juri Linkov
2022-03-14  9:08                 ` Ergus
2022-03-14 18:19                   ` Juri Linkov
2022-03-14 19:46                     ` Ergus
2022-03-14 20:58                       ` Stefan Monnier
2022-03-14 22:28                         ` Ergus
2022-03-14 22:34                           ` Stefan Monnier
2022-03-15 15:22                           ` Stefan Monnier
2022-03-14 19:34                   ` Juri Linkov
2022-03-17 18:01                     ` Ergus
2022-03-17 18:47                     ` Ergus
2022-03-17 20:56                       ` Juri Linkov
2022-03-17 21:26                         ` Stefan Monnier
2022-03-17 21:33                           ` Ergus
2022-03-17 22:44                             ` Stefan Monnier
2022-03-17 22:44                         ` Ergus [this message]
2022-03-18  6:25                           ` Eli Zaretskii
2022-03-17 23:10                         ` Ergus
2022-03-18  6:28                           ` Eli Zaretskii
2022-03-18 11:16                             ` Ergus
2022-03-18 11:33                               ` Eli Zaretskii
2022-03-18 12:04                             ` Stefan Monnier
2022-03-18 12:11                               ` Eli Zaretskii
2022-03-18 12:23                                 ` Ergus
2022-03-18 12:35                                   ` Stefan Monnier
2022-03-18 12:38                                     ` Ergus
2022-03-18 12:57                                       ` Eli Zaretskii
2022-03-18  9:27                           ` Juri Linkov
2022-03-18 11:26                             ` Eli Zaretskii
2022-03-18 11:53                             ` Ergus
2022-03-18 12:06                               ` Eli Zaretskii
2022-03-18 21:31                             ` Ergus
2022-03-19 19:03                               ` Juri Linkov
2022-03-19 20:43                                 ` Ergus
2022-03-19 20:46                                 ` Ergus
2022-03-20 18:42                                   ` Juri Linkov
2022-03-20 22:00                                     ` Ergus
2022-03-21  8:32                                       ` Juri Linkov
2022-03-18 12:08                           ` Stefan Monnier
2022-03-18 12:13                             ` Eli Zaretskii
2022-03-12  0:14       ` Ergus
2022-03-12 17:04         ` Lars Ingebrigtsen
2022-03-12 17:29           ` Eli Zaretskii
2022-03-12 17:37             ` Ergus
2022-03-12 18:14               ` Eli Zaretskii
2022-03-12 19:30                 ` Ergus
2022-03-12 19:39                   ` Eli Zaretskii
2022-03-13 18:55                     ` Ergus
2022-03-13 17:47                   ` Juri Linkov
2022-03-13 18:52                     ` Ergus
2022-03-12 19:11         ` [External] : " Drew Adams
2022-03-10 19:55   ` Drew Adams
2022-03-10 19:59     ` Juri Linkov
2022-03-10 23:13       ` Drew Adams
2022-03-10 19:58 ` Eli Zaretskii
2022-03-10 20:23   ` Lars Ingebrigtsen
2022-03-10 20:38     ` Eli Zaretskii
2022-03-12 18:52       ` Juri Linkov
2022-03-12 19:16         ` Eli Zaretskii

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220317224433.oiocm6da6tzmrq6e@Ergus \
    --to=spacibba@aol.com \
    --cc=emacs-devel@gnu.org \
    --cc=juri@linkov.net \
    --cc=larsi@gnus.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.