Closed flupe closed 2 years ago
@gmoe Thank you very much for taking the time to review my PR! I'll respond to your comments as I go about fixing them. I'm hoping to hear back from @munshkr as well.
Hi @flupe ! Thank you for sending this PR. I agree with your proposal of making tmux optional, now that we have proper support of the terminal in both Vim 8 and Neovim. I'll test the tmux integration in a minute and comment if something else comes up.
This is looking great!
@flupe @munshkr could either of you give a summary of where this is at, or what work remains?
I love tmux
, but this PR certainly seems like a much more conventional approach.
Thanks again for your work on this @flupe! Closing this PR as this work has landed as a part of #74.
As disccused in #38 , this PR adds support for the Vim8 Terminal feature. This allows the user to discard tmux entirely and have everything handled inside Vim. Before merging this PR, a few things need to be taken care of:
Further testing:
However, we still need to check whether my code did not break the existing tmux solution nor the Neovim terminal feature.
Perhaps we should change the wording of the README a bit. With native terminal support in both Neovim and Vim, I don't think we should emphasize tmux support this much --- we should explain how this plugin works with no dependencies, but that we have optional support for tmux as well. I actually changed the routine inside the code so that the native terminal feature is the default choice for the plugin. If a user wants to use tmux, they can effectively do:
Likewise, since we now support native terminal for both Vim8 and Neovim, I think we should ditch the custom bash scripts altogether, or at the very least only refer to them in the tmux section. The installation should be as simple as possible, only requiring a simple
Right now, having tmux as a dependency and requiring to run the makefile is too much.
That's all for now, I might think of other things later. Hope it's useful to some, don't hesitate to review, test or criticize the above.