neovim

commit 703f4037c475ee504b71d393e152fb1ae44c7bb9
parent ce0c0c31a08975dd2e34207afb1a2cd544775bad
Author: luukvbaal <luukvbaal@gmail.com>
Date:   Thu, 27 Mar 2025 12:52:46 +0100

fix(ui): wincmd _ should not increase 'cmdheight' above 0 (#33056)


Diffstat:
M
src/nvim/window.c
 | 
10
+++++-----
M
test/functional/ui/cmdline_spec.lua
 | 
6
++++++

2 files changed, 11 insertions(+), 5 deletions(-)
diff --git a/src/nvim/window.c b/src/nvim/window.c
@@ -196,6 +196,10 @@ win_T *swbuf_goto_win_with_buf(buf_T *buf)
   return wp;
 }
 
+// 'cmdheight' value explicitly set by the user: window commands are allowed to
+// resize the topframe to values higher than this minimum, but not lower.
+static OptInt min_set_ch = 1;
+
 /// all CTRL-W window commands are handled here, called from normal_cmd().
 ///
 /// @param xchar  extra char from ":wincmd gx" or NUL
@@ -513,7 +517,7 @@ newwindow:
   // set current window height
   case Ctrl__:
   case '_':
-    win_setheight(Prenum ? Prenum : Rows - 1);
+    win_setheight(Prenum ? Prenum : Rows - (int)min_set_ch);
     break;
 
   // increase current window width
@@ -3505,10 +3509,6 @@ static bool is_bottom_win(win_T *wp)
   return true;
 }
 
-// 'cmdheight' value explicitly set by the user: window commands are allowed to
-// resize the topframe to values higher than this minimum, but not lower.
-static OptInt min_set_ch = 1;
-
 /// Set a new height for a frame.  Recursively sets the height for contained
 /// frames and windows.  Caller must take care of positions.
 ///
diff --git a/test/functional/ui/cmdline_spec.lua b/test/functional/ui/cmdline_spec.lua
@@ -1352,6 +1352,12 @@ describe('cmdline height', function()
     -- cmdheight unchanged.
     eq(1, eval('&cmdheight'))
   end)
+
+  it('not increased to 0 from 1 with wincmd _', function()
+    command('set cmdheight=0 laststatus=0')
+    command('wincmd _')
+    eq(0, eval('&cmdheight'))
+  end)
 end)
 
 describe('cmdheight=0', function()