You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
Oftentimes I find myself selecting the wrong textobject (eg, vac instead of vax) or just needing to go back to the place my cursor was before I performed the textobject selection. There's no instant way to go back. You have to use vim motions to go to your original position. This is a bit inconvenient because textobjects from wellle/targets.vim add the cursor position to the jumplist just like set_jumps for the move module. So from time to time that muscle memory kicks in and you go back in the jumplist only to find yourself not where you'd expect.
Describe the solution you'd like
Add the option set_jumps, which is present in the move module to the select module. It should default to false for backward compatibility.
Describe alternatives you've considered
None, currently.
Additional context
Textobjects from wellle/targets.vim have the same jumplist functionality enabled by default for a few textobjects, but can optionally be enabled for all of them.
The implementation in targets.vimhere and the move module here are functionally the same, so we can reuse the same implementation in select and have consistency between the two plugins.
I can prepare a PR if there's interest in this functionality.
The text was updated successfully, but these errors were encountered:
Is your feature request related to a problem? Please describe.
Oftentimes I find myself selecting the wrong textobject (eg,
vac
instead ofvax
) or just needing to go back to the place my cursor was before I performed the textobject selection. There's no instant way to go back. You have to use vim motions to go to your original position. This is a bit inconvenient because textobjects fromwellle/targets.vim
add the cursor position to the jumplist just likeset_jumps
for themove
module. So from time to time that muscle memory kicks in and you go back in the jumplist only to find yourself not where you'd expect.Describe the solution you'd like
Add the option
set_jumps
, which is present in themove
module to theselect
module. It should default tofalse
for backward compatibility.Describe alternatives you've considered
None, currently.
Additional context
Textobjects from
wellle/targets.vim
have the same jumplist functionality enabled by default for a few textobjects, but can optionally be enabled for all of them.The implementation in
targets.vim
here and themove
module here are functionally the same, so we can reuse the same implementation inselect
and have consistency between the two plugins.I can prepare a PR if there's interest in this functionality.
The text was updated successfully, but these errors were encountered: