emacs-diffs
[Top][All Lists]
Advanced

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

emacs-28 1e57826: Fix control buffer window height in 'ediff-setup-contr


From: Martin Rudalics
Subject: emacs-28 1e57826: Fix control buffer window height in 'ediff-setup-control-buffer' (Bug#49277)
Date: Wed, 15 Dec 2021 12:55:39 -0500 (EST)

branch: emacs-28
commit 1e578267fb19208504d28253e0c892ceb9a34fb4
Author: Martin Rudalics <rudalics@gmx.at>
Commit: Martin Rudalics <rudalics@gmx.at>

    Fix control buffer window height in 'ediff-setup-control-buffer' (Bug#49277)
    
    * lisp/vc/ediff-util.el (ediff-setup-control-buffer): Emulate
    original 'shrink-window-if-larger-than-buffer' call by calling
    'fit-window-to-buffer' with an appropriate maximum height
    argument (Bug#49277).
---
 lisp/vc/ediff-util.el | 20 +++++++++++++++++++-
 1 file changed, 19 insertions(+), 1 deletion(-)

diff --git a/lisp/vc/ediff-util.el b/lisp/vc/ediff-util.el
index 7c36291..c12de02 100644
--- a/lisp/vc/ediff-util.el
+++ b/lisp/vc/ediff-util.el
@@ -521,7 +521,25 @@ to invocation.")
     (erase-buffer)
     (ediff-set-help-message)
     (insert ediff-help-message)
-    (shrink-window-if-larger-than-buffer)
+    ;; With the fix for Bug#49277 and an 'ediff-setup-windows-plain'
+    ;; layout, the window of the control buffer we want to adjust here
+    ;; is no longer the lower of two windows on their frame both showing
+    ;; that control buffer but rather the bottom-most window in the
+    ;; established ediff layout for that frame.  As a consequence,
+    ;; 'shrink-window-if-larger-than-buffer' will fail to show the whole
+    ;; buffer with 'ediff-toggle-help' because that window's maximum
+    ;; height is not half the height of its frame but the height of the
+    ;; control buffer's window in the established layout (Bug#52504).
+    ;;
+    ;; The form below is an attempt to emulate the behavior of Emacs 27
+    ;; as faithfully as possible in this regard (the use of 'ceiling'
+    ;; mimics the behavior of 'split-window' giving the lower window the
+    ;; residue line when the window to split has an uneven number of
+    ;; lines).
+    (when (and (window-combined-p)
+              (pos-visible-in-window-p (point-min)))
+      (fit-window-to-buffer
+       nil (ceiling (/ (window-total-height (frame-root-window)) 2.0))))
     (or (ediff-multiframe-setup-p)
        (ediff-indent-help-message))
     (ediff-set-help-overlays)



reply via email to

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