unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#14147: 24.3.50; turn on `minibuffer-depth-indicator-mode' by default
@ 2013-04-05 17:03 Drew Adams
  2013-04-05 21:38 ` Stefan Monnier
  2016-04-28 23:20 ` Lars Ingebrigtsen
  0 siblings, 2 replies; 6+ messages in thread
From: Drew Adams @ 2013-04-05 17:03 UTC (permalink / raw)
  To: 14147

It is unfortunate that this minor mode is still not turned on by
default.  There is no reason not to do so.  The mode has no effect when
there is no recursive minibuffer, which is most of the time.  And when
there is a recursive minibuffer it is essential that that fact be
communicated to users, one way or another.  This mode provides one way.
 
mb-depth.el has been present in the code for years now, but it is still
essentially ignored, unknown to most users.  Please put it to use,
letting users know when they are in a recursive minibuffer, by default.
Novice users are the least likely to discover this feature, and they are
perhaps the most in need of its aid.
 
This is no different from Emacs automatically adding nested `[...]' to
the mode line to indicate recursive editing levels.  No one would
suggest that that indicator be turned off by default and that users be
forced to learn about it on their own and turn it on.  The same should
be the case for indicating recursive minibuffer levels.  Should be a
no-brainer.

In GNU Emacs 24.3.50.1 (i386-mingw-nt5.1.2600)
 of 2013-03-28 on ODIEONE
Bzr revision: 112173 monnier@iro.umontreal.ca-20130328163306-419khox8aneaoaxv
Windowing system distributor `Microsoft Corp.', version 5.1.2600
Configured using:
 `configure --with-gcc (4.7) --no-opt --enable-checking --cflags
 -IC:/Devel/emacs/build/include --ldflags -LC:/Devel/emacs/build/lib'
 






^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#14147: 24.3.50; turn on `minibuffer-depth-indicator-mode' by default
  2013-04-05 17:03 bug#14147: 24.3.50; turn on `minibuffer-depth-indicator-mode' by default Drew Adams
@ 2013-04-05 21:38 ` Stefan Monnier
  2013-04-06  0:19   ` Drew Adams
  2016-04-28 23:20 ` Lars Ingebrigtsen
  1 sibling, 1 reply; 6+ messages in thread
From: Stefan Monnier @ 2013-04-05 21:38 UTC (permalink / raw)
  To: Drew Adams; +Cc: 14147

> there is a recursive minibuffer it is essential that that fact be
> communicated to users, one way or another.  This mode provides one way.

It is communicated, by default: it signals an error.


        Stefan





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#14147: 24.3.50; turn on `minibuffer-depth-indicator-mode' by default
  2013-04-05 21:38 ` Stefan Monnier
@ 2013-04-06  0:19   ` Drew Adams
  0 siblings, 0 replies; 6+ messages in thread
From: Drew Adams @ 2013-04-06  0:19 UTC (permalink / raw)
  To: 'Stefan Monnier'; +Cc: 14147

> > when there is a recursive minibuffer it is essential that
> > that fact be communicated to users, one way or another.
> > This mode provides one way.
> 
> It is communicated, by default: it signals an error.

That's your answer?

Why not turn on `minibuffer-depth-indicator-mode' by default?  That's the
question, not whether you signal that there is a recursive minibuffer.

Or if that's really your answer then why not also signal an error whenever there
is a recursive edit, instead of showing `[...]' in the minibuffer?

(To be clear, FWIW, this is not for me or anyone who uses my code.  Icicle mode
has always turned on `minibuffer-depth-indicator-mode'.)






^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#14147: 24.3.50; turn on `minibuffer-depth-indicator-mode' by default
  2013-04-05 17:03 bug#14147: 24.3.50; turn on `minibuffer-depth-indicator-mode' by default Drew Adams
  2013-04-05 21:38 ` Stefan Monnier
@ 2016-04-28 23:20 ` Lars Ingebrigtsen
  2016-04-29 16:46   ` Drew Adams
  1 sibling, 1 reply; 6+ messages in thread
From: Lars Ingebrigtsen @ 2016-04-28 23:20 UTC (permalink / raw)
  To: Drew Adams; +Cc: 14147

"Drew Adams" <drew.adams@oracle.com> writes:

> It is unfortunate that this minor mode is still not turned on by
> default.  There is no reason not to do so.  The mode has no effect when
> there is no recursive minibuffer, which is most of the time.

I guess there's little point in having a global mode switched on when
virtually no users will be using it.  Having it on would mean Emacs
should be dumped with mb-depth.el, and I don't see that happening.

However, I think everybody who uses `enable-recursive-minibuffers' would
probably prefer that it's on.  I've now mentioned this mode in the doc
string of the latter.

Closing.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#14147: 24.3.50; turn on `minibuffer-depth-indicator-mode' by default
  2016-04-28 23:20 ` Lars Ingebrigtsen
@ 2016-04-29 16:46   ` Drew Adams
  2016-05-02  9:28     ` Nicolas Richard
  0 siblings, 1 reply; 6+ messages in thread
From: Drew Adams @ 2016-04-29 16:46 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: 14147

> > It is unfortunate that this minor mode is still not turned on by
> > default.  There is no reason not to do so.  The mode has no effect when
> > there is no recursive minibuffer, which is most of the time.
> 
> I guess there's little point in having a global mode switched on when
> virtually no users will be using it.

Why the presumption that "no users will be using it"?

> Having it on would mean Emacs should be dumped with mb-depth.el,
> and I don't see that happening.

IOW, you don't want it on by default.

> However, I think everybody who uses `enable-recursive-minibuffers' would
> probably prefer that it's on.  I've now mentioned this mode in the doc
> string of the latter.
> 
> Closing.

It's not about users who have _customized_
`enable-recursive-minibuffers' to non-nil.

I imagine that few, if any, users do that.  But some commands do
BIND that var to non-nil to allow prompting for something else
during use of the minibuffer.

As the bug report said:

  This is no different from Emacs automatically adding nested
  `[...]' to the mode line to indicate recursive editing levels.

  No one would suggest that that indicator be turned off by default
  and that users be forced to learn about it on their own and turn
  it on.  The same should be the case for indicating recursive
  minibuffer levels.  Should be a no-brainer.

Perhaps you would argue that, because "no users will be using"
recursive edits, Emacs should stop automatically adding `[...]'
to the mode line to indicate recursive editing levels?  Make
users request that feature explicitly, perhaps by loading a
library?





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#14147: 24.3.50; turn on `minibuffer-depth-indicator-mode' by default
  2016-04-29 16:46   ` Drew Adams
@ 2016-05-02  9:28     ` Nicolas Richard
  0 siblings, 0 replies; 6+ messages in thread
From: Nicolas Richard @ 2016-05-02  9:28 UTC (permalink / raw)
  To: Drew Adams; +Cc: 14147, Lars Ingebrigtsen

> It's not about users who have _customized_
> `enable-recursive-minibuffers' to non-nil.
>
> I imagine that few, if any, users do that.

I disagree with that part, from my own experience : I tried to use
recursive minibuffers, but emacs complained, so I customized the
variable. However I did not enable minibuffer-depth-indicate-mode until
much later when I finally found about it...

Thus I too think it should be on by default, but having it in the
docstring is better than nothing.

-- 
Nicolas





^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2016-05-02  9:28 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-05 17:03 bug#14147: 24.3.50; turn on `minibuffer-depth-indicator-mode' by default Drew Adams
2013-04-05 21:38 ` Stefan Monnier
2013-04-06  0:19   ` Drew Adams
2016-04-28 23:20 ` Lars Ingebrigtsen
2016-04-29 16:46   ` Drew Adams
2016-05-02  9:28     ` Nicolas Richard

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).