fix(ALL): make floating windows work with string array 'winborder'

echasnovski · echasnovski · commit 890f6a3972a3 · 2025-11-11T10:31:32.000+02:00
Details:
- Neovim&gt;=0.12 allows 'winborder' to be a comma-separated list of
  characters (to be used as border characters).
  However, this is *not* a valid value of `border` for `nvim_open_win()`
  and `nvim_win_set_config()` (see Neovim's issue number 35065).

  The solution here is to not explicitly set `vim.o.winborder` as
  `border` field of floating window config. Instead, omit setting it at
  all if 'winborder' is not default (in which case it will be used as a
  fallback).

  The alternative is to explicitly split comma-separated string into Lua
  array, but there is usually no good way to do that concisely. However,
  it would allow supporting this type of border value in a per-module
  border value (which is somewhat nice to have, but deviates from Neovim
  core's decisions).
diff --git a/lua/mini/clue.lua b/lua/mini/clue.lua
@@ -1689,7 +1689,7 @@ H.window_get_config = function()
     col = vim.o.columns,
     height = math.min(vim.api.nvim_buf_line_count(buf_id), max_height),
     title = ' ' .. title .. ' ',
-    border = (vim.fn.exists('+winborder') == 1 and vim.o.winborder ~= '') and vim.o.winborder or 'single',
+    border = (vim.fn.exists('+winborder') == 0 or vim.o.winborder == '') and 'single' or nil,
   }
   local user_config = H.expand_callable(H.get_config().window.config, buf_id) or {}
   local res = vim.tbl_deep_extend('force', H.default_win_config, cur_config_fields, user_config)
diff --git a/lua/mini/files.lua b/lua/mini/files.lua
@@ -1935,7 +1935,7 @@ H.explorer_show_help = function(explorer, explorer_buf_id, explorer_win_id)
   config.height = #lines
   config.title = " 'mini.files' help "
   config.zindex = config.zindex + 1
-  local default_border = (vim.fn.exists('+winborder') == 1 and vim.o.winborder ~= '') and vim.o.winborder or 'single'
+  local default_border = (vim.fn.exists('+winborder') == 0 or vim.o.winborder == '') and 'single' or nil
   config.border = config.border or default_border
   config.style = 'minimal'
 
@@ -2411,7 +2411,7 @@ end
 H.window_open = function(buf_id, config)
   -- Add always the same extra data
   config.anchor = 'NW'
-  config.border = (vim.fn.exists('+winborder') == 1 and vim.o.winborder ~= '') and vim.o.winborder or 'single'
+  config.border = (vim.fn.exists('+winborder') == 0 or vim.o.winborder == '') and 'single' or nil
   config.focusable = true
   config.relative = 'editor'
   config.style = 'minimal'
diff --git a/lua/mini/misc.lua b/lua/mini/misc.lua
@@ -684,7 +684,7 @@ MiniMisc.zoom = function(buf_id, config)
   local compute_config = function()
     -- Use precise dimensions for no Command line interactions (better scroll)
     local max_width, max_height = vim.o.columns, vim.o.lines - vim.o.cmdheight
-    local default_border = (vim.fn.exists('+winborder') == 1 and vim.o.winborder ~= '') and vim.o.winborder or 'none'
+    local default_border = (vim.fn.exists('+winborder') == 0 or vim.o.winborder == '') and 'none' or nil
     --stylua: ignore
     local default_config = {
       relative = 'editor', row = 0, col = 0,
diff --git a/lua/mini/notify.lua b/lua/mini/notify.lua
@@ -806,7 +806,7 @@ H.window_compute_config = function(buf_id, is_for_open)
   local default_config = { relative = 'editor', style = 'minimal', noautocmd = is_for_open, zindex = 999 }
   default_config.anchor, default_config.col, default_config.row = 'NE', vim.o.columns, has_tabline and 1 or 0
   default_config.width, default_config.height = H.buffer_default_dimensions(buf_id, config_win.max_width_share)
-  default_config.border = (vim.fn.exists('+winborder') == 1 and vim.o.winborder ~= '') and vim.o.winborder or 'single'
+  default_config.border = (vim.fn.exists('+winborder') == 0 or vim.o.winborder == '') and 'single' or nil
   default_config.title = ' Notifications '
   -- Don't allow focus to not disrupt window navigation
   default_config.focusable = false
diff --git a/lua/mini/pick.lua b/lua/mini/pick.lua
@@ -2281,14 +2281,15 @@ H.picker_compute_win_config = function(win_config, is_for_open)
   local max_height = vim.o.lines - vim.o.cmdheight - (has_tabline and 1 or 0) - (has_statusline and 1 or 0)
   local max_width = vim.o.columns
 
+  local winborder = vim.fn.exists('+winborder') == 0 and '' or vim.o.winborder
   local default_config = {
     relative = 'editor',
     anchor = 'SW',
     width = math.floor(0.618 * max_width),
     height = math.floor(0.618 * max_height),
     col = 0,
     row = max_height + (has_tabline and 1 or 0),
-    border = (vim.fn.exists('+winborder') == 1 and vim.o.winborder ~= '') and vim.o.winborder or 'single',
+    border = winborder == '' and 'single' or nil,
     style = 'minimal',
     noautocmd = is_for_open,
     -- Use high enough value to be on top of built-in windows (pmenu, etc.)
@@ -2297,7 +2298,7 @@ H.picker_compute_win_config = function(win_config, is_for_open)
   local config = vim.tbl_deep_extend('force', default_config, H.expand_callable(win_config) or {})
 
   -- Tweak config values to ensure they are proper
-  if config.border == 'none' then config.border = { '', ' ', '', '', '', ' ', '', '' } end
+  if (config.border or winborder) == 'none' then config.border = { '', ' ', '', '', '', ' ', '', '' } end
   -- - Adjust dimensions accounting for actually present border parts
   local bor, n = config.border, type(config.border) == 'table' and #config.border or 0
   local height_offset = n == 0 and 2 or ((bor[1 % n + 1] == '' and 0 or 1) + (bor[5 % n + 1] == '' and 0 or 1))
diff --git a/lua/mini/test.lua b/lua/mini/test.lua
@@ -2060,7 +2060,7 @@ H.buffer_reporter.default_window_opts = function()
     height = math.floor(0.618 * vim.o.lines),
     row = math.floor(0.191 * vim.o.lines),
     col = math.floor(0.191 * vim.o.columns),
-    border = (vim.fn.exists('+winborder') == 1 and vim.o.winborder ~= '') and vim.o.winborder or 'single',
+    border = (vim.fn.exists('+winborder') == 0 or vim.o.winborder == '') and 'single' or nil,
     title = ' Test results ',
   }
 end
diff --git a/tests/screenshots/tests-test_clue.lua---Showing-keys---respects-'winborder'-option-003 b/tests/screenshots/tests-test_clue.lua---Showing-keys---respects-'winborder'-option-003
@@ -0,0 +1,23 @@
+--|---------|---------|---------|---------|
+01|                                        
+02|~                                       
+03|~                                       
+04|~                                       
+05|~                                       
+06|~       + <Space> ---------------------+
+07|~       | a │ LHS: " a"                |
+08|~       +------------------------------+
+09|[No Name] 0,0-1                         
+10|                                        
+
+--|---------|---------|---------|---------|
+01|0000000000000000000000000000000000000000
+02|1111111111111111111111111111111111111111
+03|1111111111111111111111111111111111111111
+04|1111111111111111111111111111111111111111
+05|1111111111111111111111111111111111111111
+06|1111111123333333332222222222222222222222
+07|1111111124445666666666666666666666666662
+08|1111111122222222222222222222222222222222
+09|7777777777777777777777777777777777777777
+10|8888888888888888888888888888888888888888
diff --git a/tests/screenshots/tests-test_files.lua---Windows---respect-'winborder'-option-003 b/tests/screenshots/tests-test_files.lua---Windows---respect-'winborder'-option-003
@@ -0,0 +1,33 @@
+--|---------|---------|---------|---------|
+01|+MOCK_ROOT/tests/dir-files/common -----+
+02|| .a-dir                              |
+03|| a-dir                               |
+04|| b-dir                               |
+05|| .a-file                             |
+06|| a-file                              |
+07|| A-file-2                            |
+08|| b-file                              |
+09|+--------------------------------------+
+10|~                                       
+11|~                                       
+12|~                                       
+13|~                                       
+14|~                                       
+15|                      1,9-7         All 
+
+--|---------|---------|---------|---------|
+01|0111111111111111111111111111111111000000
+02|0222222223333333333333333333333333333330
+03|0444444455555555555555555555555555555550
+04|0444444455555555555555555555555555555550
+05|0555555555555555555555555555555555555550
+06|0555555555555555555555555555555555555550
+07|0555555555555555555555555555555555555550
+08|0555555555555555555555555555555555555550
+09|0000000000000000000000000000000000000000
+10|6666666666666666666666666666666666666666
+11|6666666666666666666666666666666666666666
+12|6666666666666666666666666666666666666666
+13|6666666666666666666666666666666666666666
+14|6666666666666666666666666666666666666666
+15|7777777777777777777777777777777777777777
diff --git a/tests/screenshots/tests-test_files.lua---show_help()---respects-'winborder'-option b/tests/screenshots/tests-test_files.lua---show_help()---respects-'winborder'-option
@@ -0,0 +1,43 @@
+--|---------|---------|---------|---------|
+01|╭╭ 'mini.files' help ────────────╮─────╮
+02|││Buffer mappings:               │     │
+03|││                               │     │
+04|││Close                    │ q   │     │
+05|││Go in entry              │ l   │     │
+06|││Go in entry plus         │ L   │     │
+07|││Go out of directory      │ h   │     │
+08|││Go out of directory plus │ H   │     │
+09|╰│Go to bookmark           │ '   │─────╯
+10|~│Reset                    │ <BS>│      
+11|~│Reveal cwd               │ @   │      
+12|~│Set bookmark             │ m   │      
+13|~│Show Help                │ g?  │      
+14|~│Synchronize              │ =   │      
+15|~│Trim branch left         │ <   │      
+16|~│Trim branch right        │ >   │      
+17|~│                               │      
+18|~│(Press `q` to close)           │      
+19|~╰───────────────────────────────╯      
+20|                      1,1           All 
+
+--|---------|---------|---------|---------|
+01|0011111111111111111110000000000000000000
+02|0022222222222222222222222222222220222220
+03|0033333333333333333333333333333330333330
+04|0033333333333333333333333333333330333330
+05|0033333333333333333333333333333330333330
+06|0033333333333333333333333333333330333330
+07|0033333333333333333333333333333330333330
+08|0033333333333333333333333333333330333330
+09|0033333333333333333333333333333330000000
+10|4033333333333333333333333333333330444444
+11|4033333333333333333333333333333330444444
+12|4033333333333333333333333333333330444444
+13|4033333333333333333333333333333330444444
+14|4033333333333333333333333333333330444444
+15|4033333333333333333333333333333330444444
+16|4033333333333333333333333333333330444444
+17|4033333333333333333333333333333330444444
+18|4033333333333333333333333333333330444444
+19|4000000000000000000000000000000000444444
+20|5555555555555555555555555555555555555555
diff --git a/tests/screenshots/tests-test_files.lua---show_help()---respects-'winborder'-option-002 b/tests/screenshots/tests-test_files.lua---show_help()---respects-'winborder'-option-002
@@ -0,0 +1,43 @@
+--|---------|---------|---------|---------|
+01|╔╔ 'mini.files' help ════════════╗═════╗
+02|║║Buffer mappings:               ║     ║
+03|║║                               ║     ║
+04|║║Close                    │ q   ║     ║
+05|║║Go in entry              │ l   ║     ║
+06|║║Go in entry plus         │ L   ║     ║
+07|║║Go out of directory      │ h   ║     ║
+08|║║Go out of directory plus │ H   ║     ║
+09|╚║Go to bookmark           │ '   ║═════╝
+10|~║Reset                    │ <BS>║      
+11|~║Reveal cwd               │ @   ║      
+12|~║Set bookmark             │ m   ║      
+13|~║Show Help                │ g?  ║      
+14|~║Synchronize              │ =   ║      
+15|~║Trim branch left         │ <   ║      
+16|~║Trim branch right        │ >   ║      
+17|~║                               ║      
+18|~║(Press `q` to close)           ║      
+19|~╚═══════════════════════════════╝      
+20|                      1,1           All 
+
+--|---------|---------|---------|---------|
+01|0011111111111111111110000000000000000000
+02|0022222222222222222222222222222220222220
+03|0033333333333333333333333333333330333330
+04|0033333333333333333333333333333330333330
+05|0033333333333333333333333333333330333330
+06|0033333333333333333333333333333330333330
+07|0033333333333333333333333333333330333330
+08|0033333333333333333333333333333330333330
+09|0033333333333333333333333333333330000000
+10|4033333333333333333333333333333330444444
+11|4033333333333333333333333333333330444444
+12|4033333333333333333333333333333330444444
+13|4033333333333333333333333333333330444444
+14|4033333333333333333333333333333330444444
+15|4033333333333333333333333333333330444444
+16|4033333333333333333333333333333330444444
+17|4033333333333333333333333333333330444444
+18|4033333333333333333333333333333330444444
+19|4000000000000000000000000000000000444444
+20|5555555555555555555555555555555555555555
diff --git a/tests/screenshots/tests-test_files.lua---show_help()---respects-'winborder'-option-003 b/tests/screenshots/tests-test_files.lua---show_help()---respects-'winborder'-option-003
@@ -0,0 +1,43 @@
+--|---------|---------|---------|---------|
+01|++ 'mini.files' help ------------+-----+
+02|||Buffer mappings:               |     |
+03|||                               |     |
+04|||Close                    │ q   |     |
+05|||Go in entry              │ l   |     |
+06|||Go in entry plus         │ L   |     |
+07|||Go out of directory      │ h   |     |
+08|||Go out of directory plus │ H   |     |
+09|+|Go to bookmark           │ '   |-----+
+10|~|Reset                    │ <BS>|      
+11|~|Reveal cwd               │ @   |      
+12|~|Set bookmark             │ m   |      
+13|~|Show Help                │ g?  |      
+14|~|Synchronize              │ =   |      
+15|~|Trim branch left         │ <   |      
+16|~|Trim branch right        │ >   |      
+17|~|                               |      
+18|~|(Press `q` to close)           |      
+19|~+-------------------------------+      
+20|                      1,1           All 
+
+--|---------|---------|---------|---------|
+01|0011111111111111111110000000000000000000
+02|0022222222222222222222222222222220222220
+03|0033333333333333333333333333333330333330
+04|0033333333333333333333333333333330333330
+05|0033333333333333333333333333333330333330
+06|0033333333333333333333333333333330333330
+07|0033333333333333333333333333333330333330
+08|0033333333333333333333333333333330333330
+09|0033333333333333333333333333333330000000
+10|4033333333333333333333333333333330444444
+11|4033333333333333333333333333333330444444
+12|4033333333333333333333333333333330444444
+13|4033333333333333333333333333333330444444
+14|4033333333333333333333333333333330444444
+15|4033333333333333333333333333333330444444
+16|4033333333333333333333333333333330444444
+17|4033333333333333333333333333333330444444
+18|4033333333333333333333333333333330444444
+19|4000000000000000000000000000000000444444
+20|5555555555555555555555555555555555555555
diff --git a/tests/screenshots/tests-test_misc.lua---zoom()---respects-'winborder'-option-003 b/tests/screenshots/tests-test_misc.lua---zoom()---respects-'winborder'-option-003
@@ -0,0 +1,13 @@
+-|---------|---------|---------|
+1|+ Zoom ----------------------+
+2||                            |
+3||~                           |
+4|+----------------------------+
+5|               0,0-1      All 
+
+-|---------|---------|---------|
+1|011111100000000000000000000000
+2|022222222222222222222222222220
+3|033333333333333333333333333330
+4|000000000000000000000000000000
+5|444444444444444444444444444444
diff --git a/tests/screenshots/tests-test_notify.lua---Window---respects-'winborder'-option-003 b/tests/screenshots/tests-test_notify.lua---Window---respects-'winborder'-option-003
@@ -0,0 +1,27 @@
+--|---------|---------|---------|---------|-----
+01|                           + Notifications -+
+02|~                          |12:35:00 │ Hello|
+03|~                          +----------------+
+04|~                                            
+05|~                                            
+06|~                                            
+07|~                                            
+08|~                                            
+09|~                                            
+10|~                                            
+11|~                                            
+12|                           0,0-1         All 
+
+--|---------|---------|---------|---------|-----
+01|000000000000000000000000000122222222222222211
+02|333333333333333333333333333144444444444444441
+03|333333333333333333333333333111111111111111111
+04|333333333333333333333333333333333333333333333
+05|333333333333333333333333333333333333333333333
+06|333333333333333333333333333333333333333333333
+07|333333333333333333333333333333333333333333333
+08|333333333333333333333333333333333333333333333
+09|333333333333333333333333333333333333333333333
+10|333333333333333333333333333333333333333333333
+11|333333333333333333333333333333333333333333333
+12|555555555555555555555555555555555555555555555
diff --git a/tests/screenshots/tests-test_pick.lua---Overall-view---respects-'winborder'-option-004 b/tests/screenshots/tests-test_pick.lua---Overall-view---respects-'winborder'-option-004
@@ -0,0 +1,33 @@
+--|---------|---------|---------|---------|
+01|                                        
+02|~                                       
+03|~                                       
+04|~                                       
+05|+> ▏---------------------+              
+06||a                       |              
+07||b                       |              
+08||c                       |              
+09||                        |              
+10||                        |              
+11||                        |              
+12||                        |              
+13||                        |              
+14|+ <No name> ------ 1|3|3 +              
+15|                                        
+
+--|---------|---------|---------|---------|
+01|0000000000000000000000000000000000000000
+02|1111111111111111111111111111111111111111
+03|1111111111111111111111111111111111111111
+04|1111111111111111111111111111111111111111
+05|2334222222222222222222222211111111111111
+06|2555555555555555555555555211111111111111
+07|2666666666666666666666666211111111111111
+08|2666666666666666666666666211111111111111
+09|2777777777777777777777777211111111111111
+10|2777777777777777777777777211111111111111
+11|2777777777777777777777777211111111111111
+12|2777777777777777777777777211111111111111
+13|2777777777777777777777777211111111111111
+14|2888888888882222228888888211111111111111
+15|0000000000000000000000000000000000000000
diff --git a/tests/screenshots/tests-test_test.lua---gen_reporter---buffer---test-+-args-{-'window={width=0.9-vim.o.columns,col=0.05-vim.o.columns}'-}-002 b/tests/screenshots/tests-test_test.lua---gen_reporter---buffer---test-+-args-{-'window={width=0.9-vim.o.columns,col=0.05-vim.o.columns}'-}-002
diff --git a/tests/test_clue.lua b/tests/test_clue.lua
diff --git a/tests/test_files.lua b/tests/test_files.lua
diff --git a/tests/test_misc.lua b/tests/test_misc.lua
diff --git a/tests/test_notify.lua b/tests/test_notify.lua
diff --git a/tests/test_pick.lua b/tests/test_pick.lua
diff --git a/tests/test_test.lua b/tests/test_test.lua

Original file line number	Diff line number	Diff line change
`@@ -1689,7 +1689,7 @@ H.window_get_config = function()`
`1689`	`1689`	`col = vim.o.columns,`
`1690`	`1690`	`height = math.min(vim.api.nvim_buf_line_count(buf_id), max_height),`
`1691`	`1691`	`title = ' ' .. title .. ' ',`
`1692`		`- border = (vim.fn.exists('+winborder') == 1 and vim.o.winborder ~= '') and vim.o.winborder or 'single',`
	`1692`	`+ border = (vim.fn.exists('+winborder') == 0 or vim.o.winborder == '') and 'single' or nil,`
`1693`	`1693`	`}`
`1694`	`1694`	`local user_config = H.expand_callable(H.get_config().window.config, buf_id) or {}`
`1695`	`1695`	`local res = vim.tbl_deep_extend('force', H.default_win_config, cur_config_fields, user_config)`
Original file line number	Diff line number	Diff line change
`@@ -2060,7 +2060,7 @@ H.buffer_reporter.default_window_opts = function()`
`2060`	`2060`	`height = math.floor(0.618 * vim.o.lines),`
`2061`	`2061`	`row = math.floor(0.191 * vim.o.lines),`
`2062`	`2062`	`col = math.floor(0.191 * vim.o.columns),`
`2063`		`- border = (vim.fn.exists('+winborder') == 1 and vim.o.winborder ~= '') and vim.o.winborder or 'single',`
	`2063`	`+ border = (vim.fn.exists('+winborder') == 0 or vim.o.winborder == '') and 'single' or nil,`
`2064`	`2064`	`title = ' Test results ',`
`2065`	`2065`	`}`
`2066`	`2066`	`end`