tide icon indicating copy to clipboard operation
tide copied to clipboard

add support for selectionRange command

Open ananthakumaran opened this issue 5 years ago • 2 comments

TypeScript 3.5 added selectionRange command which provides functionality similar to https://github.com/magnars/expand-region.el

ananthakumaran avatar Jun 01 '19 10:06 ananthakumaran

I saw this too in the TS3.5 announcement and was conflicted about creating an issue for it or not.

Looking at the expand-region code though, it seems to provide mode-specific implementations for each major mode it supports.

While tide is a minor-mode, wouldn't the correct thing to do, be for us to provide a tide-mode implementation for expand-region, instead of doing our own thing?

The way I see it now:

  1. we create a defun for tide-expand-region.
  2. we create a PR for expand-region to use this function when tide-mode is active.
  3. existing expand-region users won't need to setup any new or typescript-mode specific keybindings for this functionality.

Any opinions? Sounds good?

josteink avatar Jun 01 '19 10:06 josteink

I am not quite sure whether I really like the way selectionRange behaves and I love the way tide allows me to do things "on demand" instead of hijacking default actions. So properly boundling this with a mode sounds like the way to go for me.

MarcusRiemer avatar Jun 05 '19 05:06 MarcusRiemer