Open abeldekat opened 2 weeks ago
Thanks for the suggestion!
I can reproduce that indeed MiniExtra.pickers.lsp
does not add previous position to jumplist. After quick look, I didn't find a quick fix, though. I'll take a closer look.
@echasnovski,
I noticed that you changed the type of the issue from "bug" into "feature-request". Was my assumption incorrect?
I think it is reasonable to consider a bug as a deviation from documented and/or assumed during implementation behavior.
In this particular case adding current position to jumplist is neither.
I understand. Thanks!
Contributing guidelines
Module(s)
mini.pick, mini.extra
Description
https://github.com/echasnovski/mini.nvim/assets/58370433/c5362c94-b91c-44f9-822b-f0b06b999696
When using
lsp definition
, I often jump back withctrl-o
. The jump is not correct. I also notice this behavior when usinglsp references
Neovim version
0.10
Steps to reproduce
Steps using the repro above:
repro init.lua
:30
, selecting the line with "clone"gd
andenter
ctrl-o
Close neovim and change the following:
Repeat steps 1 till 6. The cursor correctly jumps to line 30.
Expected behavior
When I jump back after using
Pick lsp scope="definition"
, the jump is correct.Actual behavior
Jump is incorrect.