fix(files): don't ever show cursor line in file previews

Resolve #1391 (hopefully, for real this time)
echasnovski · Dec 12, 2024 · e78a98a · e78a98a
1 parent d084df4
commit e78a98a
Show file tree

Hide file tree

Showing 9 changed files with 61 additions and 66 deletions.
diff --git a/lua/mini/files.lua b/lua/mini/files.lua
@@ -2477,9 +2477,6 @@ H.window_update = function(win_id, config)
   -- Reset basic highlighting (removes possible "focused" highlight group)
   H.window_update_highlight(win_id, 'FloatTitle', 'MiniFilesTitle')
 
-  -- Make sure that 'cursorline' is not overridden by `config.style`
-  vim.wo[win_id].cursorline = true
-
   -- Make sure proper `conceallevel` (can be not the case with 'noice.nvim')
   vim.wo[win_id].conceallevel = 3
 end
@@ -2513,12 +2510,10 @@ H.window_set_view = function(win_id, view)
   --   should already be invalidated.
   buf_data.win_id = win_id
 
-  -- Set visible cursor for directories
-  if H.fs_get_type(buf_data.path) == 'directory' then
-    pcall(H.window_set_cursor, win_id, view.cursor)
-    -- Set 'cursorline' here also because changing buffer might have removed it
-    vim.wo[win_id].cursorline = true
-  end
+  -- Set cursor (if defined), visible only in directories
+  pcall(H.window_set_cursor, win_id, view.cursor)
+  -- NOTE: set 'cursorline' here because changing buffer might remove it
+  vim.wo[win_id].cursorline = H.fs_get_type(buf_data.path) == 'directory'
 
   -- Update border highlight based on buffer status
   H.window_update_border_hl(win_id)

diff --git a/...s.lua---Events---`MiniFilesWindowUpdate`-can-customize-internally-set-window-config-parts b/...s.lua---Events---`MiniFilesWindowUpdate`-can-customize-internally-set-window-config-parts
@@ -17,7 +17,7 @@
 
 --|---------|---------|---------|---------|---------|---------|---------|---------|
 01|01222222222001333333100000000000000000000000000000000001222222210000000000004444
-02|05555555566006666666666666666666666666666666666666666006666666666666666666607777
+02|05555555566006666666666666666666666666666666666666666001111111111111111111107777
 03|08888888111001111111111111111111111111111111111111111009999999999999999999907777
 04|08888888111009999999999999999999999999999999999999999009999999999999999999907777
 05|01111111111009999999999999999999999999999999999999999009999999999999999999907777

diff --git a/...enshots/tests-test_files.lua---Preview---can-work-after-renaming-with-small-overall-width b/...enshots/tests-test_files.lua---Preview---can-work-after-renaming-with-small-overall-width
@@ -12,12 +12,12 @@
 
 --|---------|---------|---------|---------|---------|----
 01|011111111110233333333222222222222201111000000000000444
-02|055555555550266666666666666666666206666666666666660777
-03|000000000000222222222222222222222200000000000000000777
-04|777777777777777777777777777777777777777777777777777777
-05|777777777777777777777777777777777777777777777777777777
-06|777777777777777777777777777777777777777777777777777777
-07|777777777777777777777777777777777777777777777777777777
-08|777777777777777777777777777777777777777777777777777777
-09|777777777777777777777777777777777777777777777777777777
+02|055555555550266666666666666666666207777777777777770888
+03|000000000000222222222222222222222200000000000000000888
+04|888888888888888888888888888888888888888888888888888888
+05|888888888888888888888888888888888888888888888888888888
+06|888888888888888888888888888888888888888888888888888888
+07|888888888888888888888888888888888888888888888888888888
+08|888888888888888888888888888888888888888888888888888888
+09|888888888888888888888888888888888888888888888888888888
 10|444444444444444444444444444444444444444444444444444444
diff --git a/...ots/tests-test_files.lua---Preview---can-work-after-renaming-with-small-overall-width-002 b/...ots/tests-test_files.lua---Preview---can-work-after-renaming-with-small-overall-width-002
@@ -12,12 +12,12 @@
 
 --|---------|---------|---------|---------|---------|----
 01|011111111110022222222000000000000001111111100000000333
-02|044444444440055555555555555555555005555555555555550666
-03|000000000000000000000000000000000000000000000000000666
-04|666666666666666666666666666666666666666666666666666666
-05|666666666666666666666666666666666666666666666666666666
-06|666666666666666666666666666666666666666666666666666666
-07|666666666666666666666666666666666666666666666666666666
-08|666666666666666666666666666666666666666666666666666666
-09|666666666666666666666666666666666666666666666666666666
+02|044444444440055555555555555555555006666666666666660777
+03|000000000000000000000000000000000000000000000000000777
+04|777777777777777777777777777777777777777777777777777777
+05|777777777777777777777777777777777777777777777777777777
+06|777777777777777777777777777777777777777777777777777777
+07|777777777777777777777777777777777777777777777777777777
+08|777777777777777777777777777777777777777777777777777777
+09|777777777777777777777777777777777777777777777777777777
 10|333333333333333333333333333333333333333333333333333333
diff --git a/tests/screenshots/tests-test_files.lua---Preview---does-not-highlight-big-files b/tests/screenshots/tests-test_files.lua---Preview---does-not-highlight-big-files
@@ -17,17 +17,17 @@
 
 --|---------|---------|---------|---------|---------|---------|---------|---------|
 01|01111111111111111111111111002222222000000000000000000033333333333333333333333333
-02|04444444455555555555555555006666666666666666666666666077777777777777777777777777
-03|04444455555555555555555555000000000000000000000000000077777777777777777777777777
-04|04444444455555555555555555077777777777777777777777777777777777777777777777777777
-05|04444445555555555555555555077777777777777777777777777777777777777777777777777777
-06|06666666666666666666666666077777777777777777777777777777777777777777777777777777
-07|05555555555555555555555555077777777777777777777777777777777777777777777777777777
-08|05555555555555555555555555077777777777777777777777777777777777777777777777777777
-09|00000000000000000000000000077777777777777777777777777777777777777777777777777777
-10|77777777777777777777777777777777777777777777777777777777777777777777777777777777
-11|77777777777777777777777777777777777777777777777777777777777777777777777777777777
-12|77777777777777777777777777777777777777777777777777777777777777777777777777777777
-13|77777777777777777777777777777777777777777777777777777777777777777777777777777777
-14|77777777777777777777777777777777777777777777777777777777777777777777777777777777
+02|04444444455555555555555555005555555555555555555555555066666666666666666666666666
+03|04444455555555555555555555000000000000000000000000000066666666666666666666666666
+04|04444444455555555555555555066666666666666666666666666666666666666666666666666666
+05|04444445555555555555555555066666666666666666666666666666666666666666666666666666
+06|07777777777777777777777777066666666666666666666666666666666666666666666666666666
+07|05555555555555555555555555066666666666666666666666666666666666666666666666666666
+08|05555555555555555555555555066666666666666666666666666666666666666666666666666666
+09|00000000000000000000000000066666666666666666666666666666666666666666666666666666
+10|66666666666666666666666666666666666666666666666666666666666666666666666666666666
+11|66666666666666666666666666666666666666666666666666666666666666666666666666666666
+12|66666666666666666666666666666666666666666666666666666666666666666666666666666666
+13|66666666666666666666666666666666666666666666666666666666666666666666666666666666
+14|66666666666666666666666666666666666666666666666666666666666666666666666666666666
 15|33333333333333333333333333333333333333333333333333333333333333333333333333333333
diff --git a/tests/screenshots/tests-test_files.lua---Preview---is-not-removed-when-going-out-002 b/tests/screenshots/tests-test_files.lua---Preview---is-not-removed-when-going-out-002
@@ -17,7 +17,7 @@
 
 --|---------|---------|---------|---------|---------|---------|---------|---------|
 01|01111111111111110022222000000000000222222000000000002222222200000000333333333333
-02|04444444455555550066666666777777700777777777777777007777777777777770888888888888
+02|04444444455555550066666666777777700777777777777777005555555555555550888888888888
 03|06666666777777770055555555555555500000000000000000000000000000000000888888888888
 04|04444444555555550055555555555555508888888888888888888888888888888888888888888888
 05|05555555555555550000000000000000008888888888888888888888888888888888888888888888

diff --git a/tests/screenshots/tests-test_files.lua---Preview---works-for-files b/tests/screenshots/tests-test_files.lua---Preview---works-for-files
@@ -17,12 +17,12 @@
 
 --|---------|---------|---------|---------|---------|---------|---------|---------|
 01|01111111111111111111111111002222200000000000000000000033333333333333333333333333
-02|04444444444444444444444444005555544444644444444444444077777777777777777777777777
-03|08888888888888888888888888001111188888988888888888888077777777777777777777777777
-04|088888888888888888888888880088:8889999;<<<;=;8=;;8888077777777777777777777777777
-05|088888888888888888888888880088:8889999;<<<;=;8=;;8888077777777777777777777777777
-06|08888888888888888888888888009888888888888888888888888077777777777777777777777777
-07|08888888888888888888888888000000000000000000000000000077777777777777777777777777
+02|04444444444444444444444444001111155555655555555555555077777777777777777777777777
+03|05555555555555555555555555001111155555855555555555555077777777777777777777777777
+04|05555555555555555555555555005595558888:;;;:6:56::5555077777777777777777777777777
+05|05555555555555555555555555005595558888:;;;:6:56::5555077777777777777777777777777
+06|05555555555555555555555555008555555555555555555555555077777777777777777777777777
+07|05555555555555555555555555000000000000000000000000000077777777777777777777777777
 08|00000000000000000000000000077777777777777777777777777777777777777777777777777777
 09|77777777777777777777777777777777777777777777777777777777777777777777777777777777
 10|77777777777777777777777777777777777777777777777777777777777777777777777777777777

diff --git a/tests/screenshots/tests-test_files.lua---set_branch()---works-with-file-path-in-branch b/tests/screenshots/tests-test_files.lua---set_branch()---works-with-file-path-in-branch
@@ -14,14 +14,14 @@
 
 --|---------|---------|---------|---------|
 01|0111111111111111111110022222220000333333
-02|0444444444444444444440055555555550666666
-03|0444444444444444444440000000000000666666
-04|0444444444444444444440666666666666666666
-05|0555555555555555555550666666666666666666
-06|0444444444444444444440666666666666666666
-07|0444444444444444444440666666666666666666
-08|0000000000000000000000666666666666666666
-09|6666666666666666666666666666666666666666
-10|6666666666666666666666666666666666666666
-11|6666666666666666666666666666666666666666
+02|0444444444444444444440044444444440555555
+03|0444444444444444444440000000000000555555
+04|0444444444444444444440555555555555555555
+05|0666666666666666666660555555555555555555
+06|0444444444444444444440555555555555555555
+07|0444444444444444444440555555555555555555
+08|0000000000000000000000555555555555555555
+09|5555555555555555555555555555555555555555
+10|5555555555555555555555555555555555555555
+11|5555555555555555555555555555555555555555
 12|3333333333333333333333333333333333333333
diff --git a/tests/screenshots/tests-test_files.lua---set_branch()---works-with-file-path-in-branch-002 b/tests/screenshots/tests-test_files.lua---set_branch()---works-with-file-path-in-branch-002
@@ -14,14 +14,14 @@
 
 --|---------|---------|---------|---------|
 01|0111111111111111111110022222220000000003
-02|0444444444444444444440055555555555555506
-03|0444444444444444444440000000000000000006
-04|0444444444444444444440666666666666666666
-05|0555555555555555555550666666666666666666
-06|0444444444444444444440666666666666666666
-07|0444444444444444444440666666666666666666
-08|0000000000000000000000666666666666666666
-09|6666666666666666666666666666666666666666
-10|6666666666666666666666666666666666666666
-11|6666666666666666666666666666666666666666
+02|0444444444444444444440044444444444444405
+03|0444444444444444444440000000000000000005
+04|0444444444444444444440555555555555555555
+05|0666666666666666666660555555555555555555
+06|0444444444444444444440555555555555555555
+07|0444444444444444444440555555555555555555
+08|0000000000000000000000555555555555555555
+09|5555555555555555555555555555555555555555
+10|5555555555555555555555555555555555555555
+11|5555555555555555555555555555555555555555
 12|3333333333333333333333333333333333333333