all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: martin rudalics <rudalics@gmx.at>
To: Drew Adams <drew.adams@oracle.com>, emacs-devel <emacs-devel@gnu.org>
Subject: Re: `pop-up-frames' and binding/setting user options [was: Documenting buffer display]
Date: Mon, 22 Oct 2018 11:06:57 +0200	[thread overview]
Message-ID: <5BCD9331.6020203@gmx.at> (raw)
In-Reply-To: <d7d4a283-1014-40bc-89a9-53843b589549@default>

 > And BTW, it makes perfect sense for a command
 > that acts just like an other-window command, but
 > uses other-frame instead of other-window, to
 > bind `pop-up-frames'.
 >
 > That's precisely the meaning of `pop-up-frames':
 > make other-window commands use other-frame.

No.  The call to that command might conflict with the user's intention
to usually pop up a new frame for most buffers but to not pop up a new
frame for some _specific buffer_ whose name is specified in
'display-buffer-alist'.  The binding of 'pop-up-frames' might defeat
precisely that latter intention (even if we do our best to avoid it).

 > We create user options for user convenience.
 > They are not something that should limit users.

Overriding the customization of 'display-buffer-alist' does limit the
user.

 > Did anyone tell programmers that they must always
 > bind a variable instead of passing an argument to
 > a function?  I don't think so.

On the contrary we tell programmers to not bind a variable and always
pass an argument instead.

 > What's your problem with that thread?  I guess
 > it's this code:
 >
 > (let ((pop-up-frames  t))
 >    (bookmark-jump-other-window bookmark))
 >
 > Nothing could be simpler, clearer, or more
 > elegant for a command that jumps to a bookmark
 > in another frame, IMO.

Unless it's a specific bookmark the user wants to handle specifically.

 > Now I even wonder, since you pointed to that
 > thread, whether my showing `pop-up-frames'
 > there was perhaps a catalyst for your recent
 > discouragement of `pop-up-frames' in the doc.

Quite on the contrary.  The only thing I appreciated in that thread
was what you wrote (and cited here).

martin



  parent reply	other threads:[~2018-10-22  9:06 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-10-21 16:10 `pop-up-frames' and binding/setting user options [was: Documenting buffer display] Drew Adams
2018-10-21 17:01 ` Eli Zaretskii
2018-10-22  9:06 ` martin rudalics [this message]
2018-10-22 14:16   ` Drew Adams
2018-10-22 19:15     ` martin rudalics
2018-10-22 20:42       ` Drew Adams
2018-10-23  8:58         ` martin rudalics
2018-10-23 14:13           ` Drew Adams
2018-10-23 17:35             ` Stefan Monnier
2018-10-23 18:01               ` Drew Adams
2018-10-23 18:20                 ` Stefan Monnier
2018-10-23 18:28                   ` Stefan Monnier
2018-10-23 23:05                     ` Drew Adams
2018-10-23 22:57                   ` Drew Adams
2018-10-23 18:19             ` martin rudalics

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=5BCD9331.6020203@gmx.at \
    --to=rudalics@gmx.at \
    --cc=drew.adams@oracle.com \
    --cc=emacs-devel@gnu.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.