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

Integrated terminal: Combine "toggle" and "focus" commands #8463

Closed
chrmarti opened this issue Jun 28, 2016 · 5 comments
Closed

Integrated terminal: Combine "toggle" and "focus" commands #8463

chrmarti opened this issue Jun 28, 2016 · 5 comments
Assignees

Comments

@chrmarti
Copy link
Contributor

Testing #8222

  • VSCode Version: Code - Insiders 1.3.0-insider (d0c2b89, 2016-06-28T05:04:06.083Z)

Currently I need two keyboard shortcuts to do all three:

  • show and focus terminal when not already shown ("toggle" or "focus")
  • focus terminal when already shown ("focus")
  • hide terminal when shown ("toggle")

This could be simplified by letting the "toggle" command focus the terminal when already shown but not in focus. It would then hide the terminal only when already focused.

@Tyriar Tyriar self-assigned this Jun 29, 2016
@Tyriar
Copy link
Member

Tyriar commented Jun 29, 2016

I added focus so that ctrl+backtick didn't need to be hit twice to focus an already visible terminal. Related discussion here #7540

@Tyriar Tyriar closed this as completed Jun 29, 2016
@leojkwan
Copy link

thanks @Tyriar, ran into same inefficiency, my keybindings.json extension contains

{
        "key": "ctrl+`",
        "command": "workbench.action.terminal.toggleTerminal"
    },
    {
        "key": "ctrl+`",
        "command": "workbench.action.terminal.focus",
        "when": "!terminalFocus"
    }

@calvn
Copy link

calvn commented Feb 12, 2017

I see this issue in the latest version of VSCode (v1.9.1). If the focus is not on the terminal (e.g. focus is on editor) and I have the terminal panel visible, I have to press crtl+backtick twice. Once to toggle the terminal pane off, and the second time to toggle it back on to gain focus.

@leojkwan's suggestion works so that if the terminal panel is not on focus, pressing it the first time will gain focus, and the second time will toggle off the terminal. It would be great if this behavior is enabled by default.

@ben-x9
Copy link

ben-x9 commented Mar 2, 2017

Totally agree, this issue should be reopened.

@Tyriar
Copy link
Member

Tyriar commented Mar 2, 2017

I opened a similar discussion recently in #19400

@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants