unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Juri Linkov <juri@linkov.net>
To: Drew Adams <drew.adams@oracle.com>
Cc: 38519@debbugs.gnu.org
Subject: bug#38519: minibuffer-depth-indicate-mode with enable-recursive-minibuffers
Date: Mon, 09 Dec 2019 00:48:33 +0200	[thread overview]
Message-ID: <87v9qqwi0u.fsf@mail.linkov.net> (raw)
In-Reply-To: <603be961-d5be-41df-b9a1-d0790ae70e8c@default> (Drew Adams's message of "Sun, 8 Dec 2019 09:00:32 -0800 (PST)")

> 1. I agree that the docs should mention each other.
> And the doc strings already do that:
>
> `C-h v enable-recursive-minibuffers' says:
>   Also see `minibuffer-depth-indicate-mode',
>   which may be handy if this variable is non-nil.
>
> `C-h f minibuffer-depth-indicate-mode' says:
>   This is only useful if
>   `enable-recursive-minibuffers' is non-nil.
>
> But this hasn't yet been done in the manuals:
> (elisp) `Recursive Mini' and (emacs) `Minibuffer
> Edit' cover `enable-recursive-minibuffers', but
> `minibuffer-depth-indicate-mode' is mentioned
> nowhere, so far.

So I added mention of `minibuffer-depth-indicate-mode' to
(emacs) `Minibuffer Edit'.  Not sure about
(elisp) `Recursive Mini', probably not needed in (elisp).

> 2. I agree that `minibuffer-depth-indicate-mode'
> is useful when `enable-recursive-minibuffers'
> is non-nil (generally and only).
>
> 3. I think I disagree that there should be
> some kind of hard coupling between the two,
> which would prevent users from getting
> `enable-recursive-minibuffers' without
> `minibuffer-depth-indicate-mode'.
>
> It would be OK to automatically turn OFF
> `minibuffer-depth-indicate-mode' when
> `enable-recursive-minibuffers' is nil.  But
> that's not necessary (it has no effect), so
> it's not useful.

Actually turning `m-d-i-m' OFF when `e-r-m' is nil
is not needed because `m-d-i-m' has no visual effect
when `e-r-m' is nil - there is nothing to show
as an indication of the recursive minibuffer
when recursive minibuffers are not enabled.

> I don't think we should turn `m-d-i-m' ON
> systematically when `e-r-m' is non-nil.
> That might be OK as a default behavior, but
> we shouldn't impose it in a hard-coded way,
> so that users have no recourse.

If turning `m-d-i-m' ON when `e-r-m' is non-nil
will be the default behavior, then users could
easily to turn `m-d-i-m' OFF.





  reply	other threads:[~2019-12-08 22:48 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-12-07 23:27 bug#38519: minibuffer-depth-indicate-mode with enable-recursive-minibuffers Juri Linkov
2019-12-08 17:00 ` Drew Adams
2019-12-08 22:48   ` Juri Linkov [this message]
2021-06-14 14:34     ` Lars Ingebrigtsen

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

  List information: https://www.gnu.org/software/emacs/

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

  git send-email \
    --in-reply-to=87v9qqwi0u.fsf@mail.linkov.net \
    --to=juri@linkov.net \
    --cc=38519@debbugs.gnu.org \
    --cc=drew.adams@oracle.com \
    /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 public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).