blame.nvim icon indicating copy to clipboard operation
blame.nvim copied to clipboard

Options for blame window bleeds to info popup

Open seblj opened this issue 2 months ago • 1 comments

Hi :) The rewrite is looking really good. I recently tried it out, and I saw a minor bug with the info popup when pressing i inside the the blame window. When pressing i twice, it enters the windows, and it will set cursorline, scrollbind and maybe some other options.

I tracked it down to some places where you are using vim.api.nvim_set_option_value. For example on line 150 in window_view.lua. I tried to change those places to vim.wo[winnr][0][option] = value, and it seemed to fix the problem. This is because it only sets the option on the current buffer in the current window when providing the second indexing into vim.wo.

I see that there is a lot of other places where you use vim.wo, and it might actually be vim.wo[winnr][0] that should be used. I didn't have time yet to dig into where it might be safe to do this change, but thought I would at least open an issue so you know :) I am not sure if I will have time to dig into this and make a PR. I will ofc do it if I get time for it :)

seblj avatar Apr 21 '24 17:04 seblj