Feature: Surround current line (yss)

[ranjithshegde]

Feature description

press yss to initiate add-surround for the current line.

Feature advantages

1. saves keypresses. Without this, to get the same behaviour it requires something similar to this sequence 0ys$"C-o
2. Muscle memory for many users of vim-surround

Goes without saying, thank you for your excellent work.

[kylechui]

@ranjithshegde Would you want this feature to select the entire line? Or not including leading whitespace?

[ranjithshegde]

@kylechui Thanks for taking an interest in this feature!

I dont know how other users feel, but I am fairly used to vim-surround's way of handing it. It does not include leading or trailing whitespaces. So I would prefer no leading/training whitespaces. Unless you see an advantage in including them.

[kylechui]

I've taken a look at how vim-surround does it and it seems that tpope added a whole other keymap in order to implement the feature. I wonder if it might be better for it to be a user setting, e.g. nmap ys ^ysg_.

[ranjithshegde]

Yes that works as a solution for me. Thanks.

Shall I close the issue?

[kylechui]

I think we can close this for now, but I'll try to remember to add a list of "recommended" mappings (or something like that) to the README/wiki

[kylechui]

@ranjithshegde Just as a heads up, I decided to add this feature as a default keybinding, so you can configure it in the setup table now!

[ranjithshegde]

Thank you!