[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#410: 23.0.60; display-buffer regression

From: martin rudalics
Subject: bug#410: 23.0.60; display-buffer regression
Date: Sat, 14 Jun 2008 11:40:30 +0200
User-agent: Mozilla Thunderbird 1.0 (Windows/20041206)

> The recent reimplementation of display-buffer in lisp resulted in the
> following changed behavior, which I consider a regression:
> 1. emacs -Q
> 2. M-x calendar
>    ==> The window is vertically split, the *scratch* buffer above and
>    the Calendar buffer is below in a smaller window sized to fit.
> 3. M-: (pop-to-buffer (get-buffer "*Messages*"))
>    ==> Now the *Messages* buffer is above and the Calendar buffer is
>    still below, but the windows are evenly split, i.e. the Calendar
>    window is no longer sized to fit but is too big.  Prior to the
>    reimplementation of display-buffer doing this step did not change the
>    height of the Calendar window.
> The behavior in step 3 results from the invocation of
> window--even-window-heights in the last clause of the cond in
> display-buffer, which happens because the *Messages* buffer exits but is
> not displayed.  If the sexp in step 3 contained *scratch* instead of
> *Messages*, the height of the Calendar window would not have changed.

Thanks for noticing this.

> Looking at the pre-reimplementation source of Fdisplay_buffer, it looks
> like the window heights should get evened out just as in the lisp
> reimplementation; nevertheless, this is not what happens in the above
> recipe with the older code.

It's because in the Elisp code I evened window heights whenever they
were not equal.  The original code evened heights only when the height
of the window to display BUFFER-OR-NAME was less than that of the
selected window.  I've tried to restore the original behavior.  Please
try again.

I'd like to make the window code resilient in a way that applications
like calendar can naturally set `window-size-fixed'.  As a consequence,
`display-buffer' wouldn't change the calendar window's size even it were
larger than the new window.  Setting `window-size-fixed' can currently
result in unexpected behavior.  Fixing this is non-trivial.

Also I'd like to give `split-height-threshold' and `window-min-height'
buffer-local semantics and maybe add a `window-max-height' variable so
to do `fit-window-to-buffer' and `shrink-window-if-larger-than-buffer'
implicitly during each window configuration change.  `window-min-height'
equalling `window-max-height' for a particular buffer would then imply
`window-size-fixed' for that buffer.

Eventually, a user should be able to change window configurations
arbitrarily with the calendar window always keeping its original size
(unless there's no other way, obviously).


reply via email to

[Prev in Thread] Current Thread [Next in Thread]