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: Tue, 23 Oct 2018 20:19:31 +0200 [thread overview]
Message-ID: <5BCF6633.6050103@gmx.at> (raw)
In-Reply-To: <66b0ff55-af0d-42c0-80b1-87585e4c557f@default>
> Sure, we both acknowledged that. But why would you
> choose to use an `other-frame' command for those
> specific bookmarks for which it is not TRT? That's
> the question.
Because it might be the nearest existing approximation of what the
user wants.
> Is your argument against `pop-up-frames' only that
> of a maintainer - not wanting to bother maintaining
> support? Or is it that you see it as a bad thing
> for users to be able to use `pop-up-frames'?
>
> Saying that in some case (which I haven't seen
> demonstrated yet) it doesn't do the same thing that
> passing an equivalent argument does, does not
> invalidate its usefulness. At most it would be an
> argument for not using it in those hypothetical
> problematic cases.
'pop-up-frames' cannot be considered in isolation. It is accompanied
by two additional options - 'pop-up-frame-function' which specifies
the function to call when 'pop-up-frames' is non-nil and
'pop-up-frame-alist' which specifies the parameters passed to that
function. The implementation of the latter is not guaranteed to
always do what the user wants as I now mention in the manual text and
I have no idea how to fix that. IMO 'display-buffer-pop-up-frame'
should never have used 'pop-up-frame-function' and
'pop-up-frame-alist' in the first place. This is a design flaw that
AFAICT cannot be corrected at the present stage without introducing
yet another behavioral incompatibility. And it has minor importance
because there is no need to use 'pop-up-frames' nowadays.
martin
prev parent reply other threads:[~2018-10-23 18:19 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
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 [this message]
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=5BCF6633.6050103@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.