Re: [Bug: 21.5-b29] Dired unconditionally change frame size

"Stephen J. Turnbull" <stephen(a)xemacs.org&gt; writes: OK, I see now that the old `set-window-configuration' code would restore everything ... and then un-restore the size of the root window, and let the ensuing changes sort themselves out. I didn't implement this as I figured if you restored a window configuration, you really wanted everything restored, and not some vague approximation that fits in the current frame. (This aspect of its behavior was undocumented.) In the case of the minibuffer, this is arguably annoying, so I'd like to change the minibuffer code instead of the window-configuration code. I've attached what I have right now. Does this work for you? -- Cheers =8-} Mike Friede, Völkerverständigung und überhaupt blabla diff --git a/lisp/ChangeLog b/lisp/ChangeLog --- a/lisp/ChangeLog +++ b/lisp/ChangeLog ＠＠ -1,3 +1,9 ＠＠ +2009-07-12 Mike Sperber <mike(a)xemacs.org&gt; + + * minibuf.el (read-from-minibuffer): Do not restore frame size + along with window configuration before minibuffer read. This gets + us back to historical behavior. + 2009-06-07 Aidan Kehoe <kehoea(a)parhasard.net&gt; * code-files.el (insert-file-contents): diff --git a/lisp/minibuf.el b/lisp/minibuf.el --- a/lisp/minibuf.el +++ b/lisp/minibuf.el ＠＠ -550,8 +550,12 ＠＠ ;; if we changed frames (due to surrogate minibuffer), ;; and we're still on the new frame, go back to the old one. (select-frame oframe)) - (if mconfig (set-window-configuration mconfig)) - (set-window-configuration oconfig)))) + (let ((previous-frame-width (frame-pixel-width oframe)) + (previous-frame-height (frame-pixel-height oframe))) + (if mconfig (set-window-configuration mconfig)) + (set-window-configuration oconfig) + ;; we don't want to change the frame size + (set-frame-pixel-size oframe previous-frame-width previous-frame-height))))) (defun minibuffer-max-depth-exceeded () _______________________________________________ XEmacs-Beta mailing list XEmacs-Beta(a)xemacs.org http://calypso.tux.org/cgi-bin/mailman/listinfo/xemacs-beta