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

Aerojump taints the jump list

Open m-pilia opened this issue 4 years ago • 1 comments

Hi! Thanks for sharing this plugin, I love the concept and it looks really promising! However, I think there is a problem with some (probably unintentional) side effects.

Using the plugin adds spurious entries to the jump list. The effect is evident when launching a search and then interrupting it without jumping nor inserting any character. Ideally, the jump list should be unaltered, but it is not. The problem is the usage inside the plugin of commands that have the unwanted side effect of adding an entry to the jump list.

Moreover, the call to normal! H in the function Aerojump() can cause an annoying scroll of the buffer when starting a search (the effect is prominent e.g. if the user has scrolloff set to a high value, or for people that keep the cursor in the middle of the screen).

m-pilia avatar Nov 06 '19 19:11 m-pilia

Hi, thanks for your input. If I remember correctly, I made some effort to not clutter the jumpstack, but perhaps that is only the case when you select a match and not abort the search prematurely? I might be able to add an option for folks who like to keep the cursor in the middle of the screen.

ripxorip avatar Nov 07 '19 13:11 ripxorip