Closed pokey closed 3 months ago
I guess we need to unify this:
app.exe
? that is the normal way in community so ideal if possiblewin.title
containing nvim
win.title
containing VIM MODE
(for instance on Windows if Talon detects it as app.exe: conhost.exe
)Update: we definitely can't just rely on the window title containing nvim
, as I was doing before, because that broke my vscode when I opened a file that contained nvim
in its name 😅. I updated the matcher to look for VIM MODE
as well, like we're doing for Linux, but that does mean that installing https://github.com/hands-free-vim/talon.nvim becomes a hard requirement
deprecated by https://github.com/hands-free-vim/neovim-talon/pull/24
This might be a looser matcher than we want, but works for me. Worth reviewing to ensure it's not too loose. In particular the other matchers seem to require
VIM MODE
, but that didn't appear in my window title 🤔