barrett/canola.nvim
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 3

fix: more consistent cursor position when entering a new directory (#536)

Browse source
This commit is contained in:
Steven Arcangeli 2025-01-22 15:10:41 -08:00
parent 62c5683c2e
commit c80fa5c415
1 changed files with 23 additions and 20 deletions
Download patch file Download diff file Expand all files Collapse all files

25
lua/oil/view.lua
View file

@ -258,8 +258,9 @@ local function get_first_mutable_column_col(adapter, ranges)
end end
---Force cursor to be after hidden/immutable columns ---Force cursor to be after hidden/immutable columns
local function constrain_cursor() ---@param mode false|"name"|"editable"
if not config.constrain_cursor then local function constrain_cursor(mode)
if not mode then
return return
end end
local parser = require("oil.mutator.parser") local parser = require("oil.mutator.parser")
@ -275,14 +276,12 @@ local function constrain_cursor()
local result = parser.parse_line(adapter, line, column_defs) local result = parser.parse_line(adapter, line, column_defs)
if result and result.ranges then if result and result.ranges then
local min_col local min_col
if config.constrain_cursor == "editable" then if mode == "editable" then
min_col = get_first_mutable_column_col(adapter, result.ranges) min_col = get_first_mutable_column_col(adapter, result.ranges)
elseif config.constrain_cursor == "name" then elseif mode == "name" then
min_col = result.ranges.name[1] min_col = result.ranges.name[1]
else else
error( error(string.format('Unexpected value "%s" for option constrain_cursor', mode))
string.format('Unexpected value "%s" for option constrain_cursor', config.constrain_cursor)
)
end end
if cur[2] < min_col then if cur[2] < min_col then
vim.api.nvim_win_set_cursor(0, { cur[1], min_col }) vim.api.nvim_win_set_cursor(0, { cur[1], min_col })
@ -407,7 +406,7 @@ M.initialize = function(bufnr)
callback = function() callback = function()
-- For some reason the cursor bounces back to its original position, -- For some reason the cursor bounces back to its original position,
-- so we have to defer the call -- so we have to defer the call
vim.schedule(constrain_cursor) vim.schedule_wrap(constrain_cursor)(config.constrain_cursor)
end, end,
}) })
vim.api.nvim_create_autocmd({ "CursorMoved", "ModeChanged" }, { vim.api.nvim_create_autocmd({ "CursorMoved", "ModeChanged" }, {
@ -420,7 +419,7 @@ M.initialize = function(bufnr)
return return
end end
constrain_cursor() constrain_cursor(config.constrain_cursor)
if config.preview_win.update_on_cursor_moved then if config.preview_win.update_on_cursor_moved then
-- Debounce and update the preview window -- Debounce and update the preview window
@ -675,8 +674,8 @@ local function render_buffer(bufnr, opts)
vim.schedule(function() vim.schedule(function()
for _, winid in ipairs(vim.api.nvim_list_wins()) do for _, winid in ipairs(vim.api.nvim_list_wins()) do
if vim.api.nvim_win_is_valid(winid) and vim.api.nvim_win_get_buf(winid) == bufnr then if vim.api.nvim_win_is_valid(winid) and vim.api.nvim_win_get_buf(winid) == bufnr then
-- If we're not jumping to a specific lnum, use the current lnum so we can adjust the col if jump_idx then
local lnum = jump_idx or vim.api.nvim_win_get_cursor(winid)[1] local lnum = jump_idx
local line = vim.api.nvim_buf_get_lines(bufnr, lnum - 1, lnum, true)[1] local line = vim.api.nvim_buf_get_lines(bufnr, lnum - 1, lnum, true)[1]
local id_str = line:match("^/(%d+)") local id_str = line:match("^/(%d+)")
local id = tonumber(id_str) local id = tonumber(id_str)
@ -686,9 +685,13 @@ local function render_buffer(bufnr, opts)
local name = entry[FIELD_NAME] local name = entry[FIELD_NAME]
local col = line:find(name, 1, true) or (id_str:len() + 1) local col = line:find(name, 1, true) or (id_str:len() + 1)
vim.api.nvim_win_set_cursor(winid, { lnum, col - 1 }) vim.api.nvim_win_set_cursor(winid, { lnum, col - 1 })
return
end end
end end
end end
constrain_cursor("name")
end
end end
end) end)
end end