patch 8.0.1790: 'winfixwidth' is not always respected by :close
Problem: 'winfixwidth' is not always respected by :close.
Solution: Prefer a frame without 'winfixwidth' or 'winfixheight'. (Jason
Franklin)
diff --git a/src/testdir/test_winbuf_close.vim b/src/testdir/test_winbuf_close.vim
index ed64dd7..e461861 100644
--- a/src/testdir/test_winbuf_close.vim
+++ b/src/testdir/test_winbuf_close.vim
@@ -122,3 +122,39 @@
call delete('Xtest2')
call delete('Xtest3')
endfunc
+
+" Test that ":close" will respect 'winfixheight' when possible.
+func Test_winfixheight_on_close()
+ set nosplitbelow nosplitright
+
+ split | split | vsplit
+
+ $wincmd w
+ setlocal winfixheight
+ let l:height = winheight(0)
+
+ 3close
+
+ call assert_equal(l:height, winheight(0))
+
+ %bwipeout!
+ setlocal nowinfixheight splitbelow& splitright&
+endfunc
+
+" Test that ":close" will respect 'winfixwidth' when possible.
+func Test_winfixwidth_on_close()
+ set nosplitbelow nosplitright
+
+ vsplit | vsplit | split
+
+ $wincmd w
+ setlocal winfixwidth
+ let l:width = winwidth(0)
+
+ 3close
+
+ call assert_equal(l:width, winwidth(0))
+
+ %bwipeout!
+ setlocal nowinfixwidth splitbelow& splitright&
+endfunction
diff --git a/src/version.c b/src/version.c
index 9d00e46..467b14e 100644
--- a/src/version.c
+++ b/src/version.c
@@ -762,6 +762,8 @@
static int included_patches[] =
{ /* Add new patch number below this line */
/**/
+ 1790,
+/**/
1789,
/**/
1788,
diff --git a/src/window.c b/src/window.c
index 333663b..d3ec4cd 100644
--- a/src/window.c
+++ b/src/window.c
@@ -2740,12 +2740,14 @@
}
/*
- * Find out which frame is going to get the freed up space when "win" is
- * closed.
- * if 'splitbelow'/'splitleft' the space goes to the window above/left.
- * if 'nosplitbelow'/'nosplitleft' the space goes to the window below/right.
- * This makes opening a window and closing it immediately keep the same window
- * layout.
+ * Return a pointer to the frame that will receive the empty screen space that
+ * is left over after "win" is closed.
+ *
+ * If 'splitbelow' or 'splitright' is set, the space goes above or to the left
+ * by default. Otherwise, the free space goes below or to the right. The
+ * result is that opening a window and then immediately closing it will
+ * preserve the initial window layout. The 'wfh' and 'wfw' settings are
+ * respected when possible.
*/
static frame_T *
win_altframe(
@@ -2753,20 +2755,40 @@
tabpage_T *tp) /* tab page "win" is in, NULL for current */
{
frame_T *frp;
- int b;
+ frame_T *other_fr, *target_fr;
if (tp == NULL ? ONE_WINDOW : tp->tp_firstwin == tp->tp_lastwin)
- /* Last window in this tab page, will go to next tab page. */
return alt_tabpage()->tp_curwin->w_frame;
frp = win->w_frame;
- if (frp->fr_parent != NULL && frp->fr_parent->fr_layout == FR_ROW)
- b = p_spr;
- else
- b = p_sb;
- if ((!b && frp->fr_next != NULL) || frp->fr_prev == NULL)
+
+ if (frp->fr_prev == NULL)
return frp->fr_next;
- return frp->fr_prev;
+ if (frp->fr_next == NULL)
+ return frp->fr_prev;
+
+ target_fr = frp->fr_next;
+ other_fr = frp->fr_prev;
+ if (p_spr || p_sb)
+ {
+ target_fr = frp->fr_prev;
+ other_fr = frp->fr_next;
+ }
+
+ /* If 'wfh' or 'wfw' is set for the target and not for the alternate
+ * window, reverse the selection. */
+ if (frp->fr_parent != NULL && frp->fr_parent->fr_layout == FR_ROW)
+ {
+ if (frame_fixed_width(target_fr) && !frame_fixed_width(other_fr))
+ target_fr = other_fr;
+ }
+ else
+ {
+ if (frame_fixed_height(target_fr) && !frame_fixed_height(other_fr))
+ target_fr = other_fr;
+ }
+
+ return target_fr;
}
/*