unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: martin rudalics <rudalics@gmx.at>
To: Justin Burkett <justin@burkett.cc>, 24828@debbugs.gnu.org
Subject: bug#24828: 26.0.50; Renaming of display-buffer-in-major-side-window broke which-key.el
Date: Mon, 31 Oct 2016 08:42:35 +0100	[thread overview]
Message-ID: <5816F5EB.30409@gmx.at> (raw)
In-Reply-To: <CAF5dqNJWV28XF4=qT3zcm1CygP_UYUqTLFjrAp-y9qQ1a7WH-Q@mail.gmail.com>

 > I wasn't aware that that function was internal, since it appears to
 > function like the other display-buffer actions.

Unfortunately so.  It was my fault to pretend otherwise.

 > If I remember right that
 > code was written by a contributor, and I don't remember the motivation for
 > choosing that one. If it really is internal then I will investigate
 > changing to the proper method when I have some more time.

I see.  The problem is that the structure on the right

     ;; +-------------------------+         +-------------------------+
     ;; |     regular window      |         |     regular window      |
     ;; |                         |         +------------+------------+
     ;; +------------+------------+   -->   | side-win 1 | side-win 2 |
     ;; | side-win 1 | side-win 2 |         |------------+------------|
     ;; |            |            |         |     which-key window    |
     ;; +------------+------------+         +------------+------------+

was never really supported.  I initially wanted to allow that but later
found out that I cannot reliably split side windows in both directions.
It might work with Emacs 24 in a few basic use cases but will surely
break with more complicated setups.  So you essentially will have to
fall back on

     ;; +-------------------------+
     ;; |     regular window      |
     ;; |                         |
     ;; +------+-------+----------|
     ;; | side-| side- | which-key|
     ;; | win-1| win-2 | window   |
     ;; +------+-------+----------+

or

     ;; +-------------------------+
     ;; |     regular window      |
     ;; |                         |
     ;; +----------- +------------|
     ;; | side-win-1 | which-key  |
     ;; |            | window     |
     ;; +------------+------------+

sooner or later anyway.  For this, ‘which-key’ should either supply a
side window slot value for customization and/or tell the user how to
customize ‘display-buffer-alist’ for displaying the which-key window.
Then the user can decide which windows she wants which-key to share with
(the completions window looks like a good candidate because it's
ephemeral too).

Note that side windows were initially designed as more persistent but I
can easily see that the ‘which-key’ use case might come in pretty handy
too.  Hence it should be possible to make the side-win-1 and side-win-2
windows temporarily very small in order to give the which-key window
sufficient space.

 From what I've seen, ‘which-key’ is a fine mode, very neatly written.
It should be in ELPA.  In any case, keep up the good work.

martin






  parent reply	other threads:[~2016-10-31  7:42 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-10-30 18:21 bug#24828: 26.0.50; Renaming of display-buffer-in-major-side-window broke which-key.el Óscar Fuentes
2016-10-30 19:34 ` martin rudalics
     [not found]   ` <CAF5dqNJWV28XF4=qT3zcm1CygP_UYUqTLFjrAp-y9qQ1a7WH-Q@mail.gmail.com>
2016-10-31  7:42     ` martin rudalics [this message]
2016-10-31 17:31       ` Justin Burkett
2016-11-01  7:41         ` martin rudalics
2016-10-31 18:23       ` Óscar Fuentes
2016-11-01  7:41         ` 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

  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=5816F5EB.30409@gmx.at \
    --to=rudalics@gmx.at \
    --cc=24828@debbugs.gnu.org \
    --cc=justin@burkett.cc \
    /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).