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

Add a default keybooard shortcut for toggling the integrated terminal #24086

Closed
giefferre opened this issue Apr 6, 2017 · 13 comments
Closed
Assignees
Labels
info-needed Issue requires more information from poster keybindings VS Code keybinding issues keyboard-layout Keyboard layout issues macos Issues with VS Code on MAC/OS X

Comments

@giefferre
Copy link

The keyboard shortcut for the integrated terminal is disabled by default, and now users need to manually re-enable it creating a key binding.

  • VSCode Version: 1.11.0 (1.11.0) 2017-04-05T20:49:18.310Z
  • OS Version: macOS 10.12.4 (16E195)

Steps to Reproduce:

  1. Open VSCode
  2. Press ^ + \ (or correspective keys according to the macOS keyboard shortcuts map)
@mrailton
Copy link

mrailton commented Apr 6, 2017

It appears that the cause is actually VSCode reading the kaymapping wrong. When I went to remap the function it was showing as ctrl + ` which is what it always has been, however when I attempted to update this and used that key combo it showed up as ctrl + § instead.

I've confirmed the mapping on my mac is correct for the keyboard that I'm using.

@giefferre
Copy link
Author

giefferre commented Apr 6, 2017

In my case the binding was not present at all (anymore) and I eventually had to manually set it again.

@chrmarti
Copy link
Contributor

chrmarti commented Apr 6, 2017

Which keyboard layout are you using?

@chrmarti chrmarti added the info-needed Issue requires more information from poster label Apr 6, 2017
@mrailton
Copy link

mrailton commented Apr 6, 2017 via email

@alexdima
Copy link
Member

alexdima commented Apr 6, 2017

@railto What do you mean VS Code reads the keymapping wrong? Here is an in-depth explanation of what we are doing -- https://github.com/Microsoft/vscode/wiki/Keybindings
We do not ship with keyboard layout specific default keybindings, e.g. we don't ship with keybindings optimized for the Irish keyboard layout. It is therefore possible that we lack some default keybindings or that they are unfortunate/badly placed. This can be upvoted in #1240

@giefferre That PDF is valid for a US standard English layout. For your current keyboard layout, the best way to find out default/your current keyboard shortcuts is to use the F1 list or Preferences > Keyboard Shortcuts.

Please let me know if you suspect there's anything else going wrong.

@jsynowiec
Copy link
Contributor

jsynowiec commented Apr 7, 2017

Related to #23896

For me upgrading to 1.11.1 disables the +` shortcut that should be assigned to Toggle Terminal command. Executing this command from the Command Palette works as supposed.

VS Code version: 1.11.1 (1.11.1)
OS: macOS: 10.12.4

screenshot 2017-04-07 07 45 51

Everything works fine in 1.10.2.

@giefferre
Copy link
Author

@alexandrudima this is what appears to be the default keybindings for the integrated terminal.

schermata 2017-04-07 alle 09 59 12

As @jsynowiec points out, it worked in 1.10.2 but stopped working in version 1.11.1.
Of course, when I run the 'toggle terminal' command from the Command Palette it works as expected.

schermata 2017-04-07 alle 10 02 33

As you can see I use the italian layout.

@jsynowiec
Copy link
Contributor

In #24186 @lordchancellor pointed out that ` is switched with § on Mac so this might indicate a problem with keymapping?

@jkzing
Copy link

jkzing commented Apr 7, 2017

Same issue happen to me. I tried to set Toggle Integrated Terminal keymap shortcut to
ctrl + `

shift + `

alt + ` .

And all of them are worked as expect only excepted cmd + `.

Is it because of that cmd + ` is a reserved keybinding under MacOS? It is used to toggle opened windows of an App.

@intellix
Copy link

intellix commented Apr 8, 2017

I was using: ctrl + ` previously with my Apple British Keyboard and it stopped working in the latest version.

The ` and § keys are mixed up as I see when re-binding that shortcut. Here's an example of a British layout :)

@mrailton
Copy link

mrailton commented Apr 8, 2017

@alexandrudima, the reply from @intellix explains exactly the issue I was trying to explain.

@alexdima alexdima added keyboard-layout Keyboard layout issues macos Issues with VS Code on MAC/OS X keybindings VS Code keybinding issues labels Apr 10, 2017
@alexdima
Copy link
Member

Thank you all!

It is difficult to juggle so many different keyboard layouts in a single issue, here is what I suggest we do:

@giefferre OP
You appear to be using an Italian keyboard layout. From https://support.apple.com/en-ca/HT201794 I will assume it looks like this:
image

From the picture it appears that there is no way to produce ` on an Italian keyboard layout. That's why the terminal is not bound out of the box under the Italian keyboard layout. You can upvote that we ship with keyboard layout specific default keybindings in #1240


@jsynowiec @intellix @railto It appears you are using the British keyboard layout and we have some issue with [Backslash] and [IntlBackslash] where they are somehow swapped from a label point of view. Let's track in #24153 which clearly states it is about the British keyboard layout.


@jkzing It appears you cannot bind cmd + ` due to the OS eating up that keypress. I am sorry, if the keypress doesn't reach us, there is nothing we can do.

@giefferre
Copy link
Author

Thanks @alexandrudima for your feedback.

The keyboard layout you posted is the right one, I used to toggle the integrated terminal using the CTRL+> < keys.

Upvoted the bug you linked.

@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
info-needed Issue requires more information from poster keybindings VS Code keybinding issues keyboard-layout Keyboard layout issues macos Issues with VS Code on MAC/OS X
Projects
None yet
Development

No branches or pull requests

7 participants