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

⌘R chords not working, when integrated terminal has focus #68444

Closed
avxkim opened this issue Feb 11, 2019 · 8 comments
Closed

⌘R chords not working, when integrated terminal has focus #68444

avxkim opened this issue Feb 11, 2019 · 8 comments
Assignees
Labels
*question Issue represents a question, should be posted to StackOverflow (VS Code) terminal Integrated terminal issues

Comments

@avxkim
Copy link

avxkim commented Feb 11, 2019

It's present in insiders build.

  • VSCode Version: 1.31
  • OS Version: OSX 10.14.2

Steps to Reproduce:

Open integrated terminal and focus it, then use chords ⌘R, Shift+R => you'll see it doesn't open available scripts from package.json. It stucks on ⌘R was pressed. Waiting for second key of chord..., then i press Shift+R and it just prints "R" in a terminal.

image

As you can see from a screenshot, i don't have anything else mapped on ⌘R and When condition is empty, so it should work when integrated terminal has focus, but it doesn't. It's been working in a previous version, but not in 1.31.

Does this issue occur when all extensions are disabled?: Yes/No

@vscodebot vscodebot bot added the terminal Integrated terminal issues label Feb 11, 2019
@usernamehw
Copy link
Contributor

You probably need to add command Id to the skipShell commands in Settings:

https://code.visualstudio.com/docs/editor/integrated-terminal#_forcing-key-bindings-to-pass-through-the-terminal

@avxkim
Copy link
Author

avxkim commented Feb 12, 2019

Just found an issue with this approach: #63271
Also i have to add all of these lines to my settings.json:

- editor.action.toggleTabFocusMode
- workbench.action.debug.continue
- workbench.action.debug.pause
- workbench.action.debug.restart
- workbench.action.debug.run
- workbench.action.debug.start
- workbench.action.debug.stepInto
- workbench.action.debug.stepOut
- workbench.action.debug.stepOver
- workbench.action.debug.stop
- workbench.action.firstEditorInGroup
- workbench.action.focusActiveEditorGroup
- workbench.action.focusEighthEditorGroup
- workbench.action.focusFifthEditorGroup
- workbench.action.focusFirstEditorGroup
- workbench.action.focusFourthEditorGroup
- workbench.action.focusLastEditorGroup
[...]

Can't i just add one single line, or i'm forced to add every default setting?

I've added it this way:

{
    "workbench.colorTheme": "Monokai",
    "terminal.integrated.commandsToSkipShell": [
        "npm-script.run"
    ]
}

Won't i mess up my other settings?

@usernamehw
Copy link
Contributor

  • editor.action.toggleTabFocusMode
  • workbench.action.debug.continue
  • workbench.action.debug.pause
  • workbench.action.debug.restart
  • workbench.action.debug.run
  • workbench.action.debug.start
  • workbench.action.debug.stepInto
  • workbench.action.debug.stepOut
  • workbench.action.debug.stepOver
  • workbench.action.debug.stop
  • workbench.action.firstEditorInGroup
  • workbench.action.focusActiveEditorGroup
  • workbench.action.focusEighthEditorGroup
  • workbench.action.focusFifthEditorGroup
  • workbench.action.focusFirstEditorGroup
  • workbench.action.focusFourthEditorGroup
  • workbench.action.focusLastEditorGroup
    [...]

You don't need to add all these lines.

@avxkim
Copy link
Author

avxkim commented Feb 12, 2019

@usernamehw they fixed it already?

@usernamehw
Copy link
Contributor

Fixed what, exactly?

These commands excluded by default (the big list)
"terminal.integrated.commandsToSkipShell" from your settings are added to that list.

@avxkim
Copy link
Author

avxkim commented Feb 12, 2019

@usernamehw i've read, that you needed to paste all these actions in order to override default ones for commandsToSkipShell.

@usernamehw
Copy link
Contributor

That was before vscode 1.31.0.

@Tyriar
Copy link
Member

Tyriar commented Feb 12, 2019

@Tyriar Tyriar closed this as completed Feb 12, 2019
@Tyriar Tyriar added the *question Issue represents a question, should be posted to StackOverflow (VS Code) label Feb 12, 2019
@vscodebot vscodebot bot locked and limited conversation to collaborators Mar 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
*question Issue represents a question, should be posted to StackOverflow (VS Code) terminal Integrated terminal issues
Projects
None yet
Development

No branches or pull requests

3 participants