Surround delete breaks the select -> action model

[wooden-worm]

In helix the surround delete command md doesn't respect selections, the action needs another key to specify which character to delete. This breaks the select -> action model since I can't visually confirm what's being deleted.

I propose helix to add a surround delete selection, which simply deletes the first and last characters of current selections.

Example:

"example text"

ma" => select everything
mx => delete first and last `"` => example text

[the-mikedavis]

This breaks the select -> action model since I can't visually confirm what's being deleted.

I agree md breaks select-then-act but with the proposed command that trims the ends of a selection, you still have no visual indication of what will be deleted.

I think this is better covered by #2413.

[wooden-worm]

Selections are enough for visual confirmation IMO. I also wouldn't mind using #2413 for surround delete, however #2413 would not work well if you want to surround delete multiple times.

Example:

[[["text"]]]

x => select the whole line
md md => ["text"]
md md => text

[mrkishi]

This one caught me off-guard. I expected this to work differently—more like ms* would mean select surround so we'd do ms*d and ms*c*. How about adding an me* for that? Read as match enclosing *.

[DavidJaz]

This also caught me off-guard. I also expected to be able to select the surrounding delimiters and then I could decide to delete or change them. Something like #9483.

It seems that Helix has decided against this behavior? I think the selection -> action model is really strong and should extend to something this basic as well.