unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Helmut Eller <eller.helmut@gmail.com>
To: emacs-devel@gnu.org
Subject: Re: help-window-select and info-lookup-symbol
Date: Tue, 21 Jul 2009 14:33:56 +0200	[thread overview]
Message-ID: <m24ot6qkp7.fsf@gmail.com> (raw)
In-Reply-To: 4A6439A9.6080406@gmx.at

* martin rudalics [2009-07-20 11:32+0200] writes:

> Restoring an old window configuration is awfully tricky.  Let W1 denote
> the window configuration before invoking help, W2 the configuration
> after invoking help (and after any `fit-window-to-buffer' and balancing
> stuff), and W3 the configuration when you press `q'.  A brute force
> approach (currently used by info or backtrace) simply restores W1 whe
> you press `q' which all so often drives me mad because it destroys the
> windows and window-buffer associations I have created in between.

For the debugger it's IMO the best option.

> Now theoretically, I could restore W1 if and only if the configurations
> W2 and W3 are equal.  In practice, however, this approach often failed
> here.  Hence, I tried to base the comparison on a weaker predicate of
> equality which ignored the identities of buffers displayed in other
> windows and the relative sizes of windows.  I got it to work for help
> buffers but was not able to make it work realiably with info buffers.

Comparing W2 and W3 per compare-window-configurations would probably
work as a start, but point should probably not be restored.

Here's the code that we use currently:

(defun slime-close-popup-window ()
  (when slime-popup-restore-data
    (destructuring-bind (popup-window selected-window old-buffer)
        slime-popup-restore-data
      (bury-buffer)
      (when (eq popup-window (selected-window))
        (cond ((and (not old-buffer) (not (one-window-p)))
               (delete-window popup-window))
              ((and old-buffer (buffer-live-p old-buffer))
               (set-window-buffer popup-window old-buffer))))
      (when (window-live-p selected-window)
        (select-window selected-window)))
    (kill-local-variable 'slime-popup-restore-data)))

popup-window is the window that displays the help content at W2.
selected-window was selected at W1.
old-buffer was the buffer associated with popup-window at W1.

It's far from perfect and it was a PITA to write.

Helmut.





  reply	other threads:[~2009-07-21 12:33 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-19 16:24 help-window-select and info-lookup-symbol Helmut Eller
2009-07-20  9:32 ` martin rudalics
2009-07-21 12:33   ` Helmut Eller [this message]
2009-07-21 13:27     ` martin rudalics
2009-07-21 13:51       ` Helmut Eller
2009-07-21 14:12         ` martin rudalics
2009-07-21 15:07           ` Helmut Eller
2009-07-22 10:11             ` martin rudalics
2009-07-22 10:28               ` Helmut Eller
2009-07-26 10:01                 ` martin rudalics
2009-07-26 10:39                   ` Helmut Eller
2009-07-26 11:00                     ` martin rudalics
2009-07-26 12:30                       ` Helmut Eller

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=m24ot6qkp7.fsf@gmail.com \
    --to=eller.helmut@gmail.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 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).