vimux icon indicating copy to clipboard operation
vimux copied to clipboard

Published 20 hours ago • verified publisher icon preservim

Hotfix for issue from PR #210

Open mvanderkamp opened this issue 2 years ago • 1 comments

#210 was missing some safety checks. See this comment. This PR adds guardrails to make sure that:

  • If VimuxUseNearest and VimuxRunnerQuery are both false/empty, we definitely don't try to re-use an existing pane/window.
  • If we do use a VimuxRunnerQuery, and that query happens to end up selecting the current pane (the one running vim), we won't use it.

mvanderkamp avatar Sep 19 '22 03:09 mvanderkamp

I'm going to do a few more manual tests that everything is working correctly with this, so please don't merge this yet.

mvanderkamp avatar Sep 19 '22 03:09 mvanderkamp

Okay, as far as I can tell everything is working correctly. I'm still not sure what exactly went wrong for newer tmux/neovim versions but it sounds like these guardrails do the trick.

mvanderkamp avatar Sep 25 '22 21:09 mvanderkamp