summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBram Moolenaar <Bram@vim.org>2021-03-13 14:29:05 +0100
committerBram Moolenaar <Bram@vim.org>2021-03-13 14:29:05 +0100
commit9e813b3dea94a8952b732a224fa31beba6e85973 (patch)
treeb078fb32462dc67b91820745579e79cfcfd04877
parent0756f757edba755b75e0c9c612ab2f7e673def84 (diff)
downloadvim-git-9e813b3dea94a8952b732a224fa31beba6e85973.tar.gz
patch 8.2.2595: setting 'winminheight' may cause 'lines' to changev8.2.2595
Problem: Setting 'winminheight' may cause 'lines' to change. Solution: Also take minimal height of other tabpages into account. (#7899)
-rw-r--r--src/testdir/test_options.vim21
-rw-r--r--src/version.c2
-rw-r--r--src/window.c4
3 files changed, 25 insertions, 2 deletions
diff --git a/src/testdir/test_options.vim b/src/testdir/test_options.vim
index 2595392a0..54042d42f 100644
--- a/src/testdir/test_options.vim
+++ b/src/testdir/test_options.vim
@@ -1035,6 +1035,27 @@ func Test_opt_winminheight_term()
call delete('Xwinminheight')
endfunc
+func Test_opt_winminheight_term_tabs()
+ CheckRunVimInTerminal
+
+ " The tabline should be taken into account.
+ let lines =<< trim END
+ set wmh=0 stal=2
+ split
+ split
+ split
+ split
+ tabnew
+ END
+ call writefile(lines, 'Xwinminheight')
+ let buf = RunVimInTerminal('-S Xwinminheight', #{rows: 11})
+ call term_sendkeys(buf, ":set wmh=1\n")
+ call WaitForAssert({-> assert_match('E36: Not enough room', term_getline(buf, 11))})
+
+ call StopVimInTerminal(buf)
+ call delete('Xwinminheight')
+endfunc
+
" Test for the 'winminwidth' option
func Test_opt_winminwidth()
only!
diff --git a/src/version.c b/src/version.c
index f1b823226..3981323f8 100644
--- a/src/version.c
+++ b/src/version.c
@@ -751,6 +751,8 @@ static char *(features[]) =
static int included_patches[] =
{ /* Add new patch number below this line */
/**/
+ 2595,
+/**/
2594,
/**/
2593,
diff --git a/src/window.c b/src/window.c
index a810a6af2..3e32b3759 100644
--- a/src/window.c
+++ b/src/window.c
@@ -5860,8 +5860,8 @@ win_setminheight(void)
// loop until there is a 'winminheight' that is possible
while (p_wmh > 0)
{
- room = Rows - p_ch - tabline_height();
- needed = frame_minheight(topframe, NULL);
+ room = Rows - p_ch;
+ needed = min_rows() - 1; // 1 was added for the cmdline
if (room >= needed)
break;
--p_wmh;