Support fish_vi_cursor on Terminal.app

[lilyball]

Apple Terminal.app gained support for xterm cursor escapes back in macOS Sierra 10.12. fish_vi_cursor hasn't realized this yet. I can set fish_vi_force_cursor but I shouldn't have to.

Terminal.app can be detected using $TERM_PROGRAM, which should have the value Apple_Terminal.

fish 3.3.1

[zanchey]

Any idea what the TERM_PROGRAM_VERSION environment variable is in Sierra?

[ridiculousfish]

I got it installed and see 388.1.1. I can tackle this when I get back next week.

[kkharji]

Sorry, but is alacirtty supported? I couldn't fix the block shape being active in insert mode

[ridiculousfish]

@tami5 I do not see any checks for alacritty, or any way to identify it from environment variables. You can set fish_vi_force_cursor. See faho's correction below

[faho]

Nah, Alacritty is supported since fish 3.2.0, @tami5: Update fish.

[ridiculousfish]

Shame on me, just saw it too.

[ridiculousfish]

I enabled these unconditionally for Apple_Terminal since the escapes appear to be ignored on versions before 10.12.