Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handle any open terminal window, rather than only the one this plugin opens #21

Open
BlueDrink9 opened this issue Apr 1, 2020 · 3 comments · May be fixed by #47
Open

Handle any open terminal window, rather than only the one this plugin opens #21

BlueDrink9 opened this issue Apr 1, 2020 · 3 comments · May be fixed by #47

Comments

@BlueDrink9
Copy link

#3 suggests using neoterm for REPL - which is a good idea. This plugin shouldn't do REPL. What I think it should do is be able to integrate with things that do, like neoterm or Nvim-R.

Can you please add a feature or options to allow the shortcut to hide/unhide already existing terminal windows, like neoterm windows?

For tabs with a single terminal window open, this should be simple. I am unsure what behavior should be for tabs with multiple terminal windows, or tabs with one window that is a terminal window.

@BlueDrink9
Copy link
Author

@skywind3000 is this feasible with the current setup?

@jiz4oh
Copy link

jiz4oh commented Jan 5, 2023

@skywind3000 could you consider this suggestion? it's very helpful

@jiz4oh
Copy link

jiz4oh commented Jan 6, 2023

Hi @BlueDrink9, you can try my fork, and add below configuration in your .vimrc

augroup vim-terminal-help-augroup
  autocmd!

  if has('nvim')
    autocmd TermOpen * let t:__terminal_bid__ = bufnr('')
  elseif exists('##TerminalWinOpen')
    autocmd TerminalWinOpen * let t:__terminal_bid__ = bufnr('')
  endif
augroup END

it's works fine for me. if anything works well, I will open a PR to contribute back the repo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants