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

Pasting into terminal find doesn't work on Windows #30994

Closed
levani opened this issue Jul 19, 2017 · 5 comments · Fixed by #33935
Closed

Pasting into terminal find doesn't work on Windows #30994

levani opened this issue Jul 19, 2017 · 5 comments · Fixed by #33935
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug help wanted Issues identified as good community contribution opportunities terminal General terminal issues that don't fall under another label verified Verification succeeded

Comments

@levani
Copy link

levani commented Jul 19, 2017

  • VSCode Version: 1.14.1
  • OS Version: Windows 10

Steps to Reproduce:

  1. Copy some text
  2. Focus on terminal and click ctrl+f and then ctrl+v

Search input is focused but after clicking ctrl+v the text is pasted in terminal itself.

Reproduces without extensions: Yes

@vscodebot vscodebot bot added the terminal General terminal issues that don't fall under another label label Jul 19, 2017
@Tyriar Tyriar added bug Issue identified by VS Code Team member as probable bug windows VS Code on Windows issues labels Jul 19, 2017
@Tyriar Tyriar changed the title ctr + f & ctrl + v pastes the text in terminal not in search box Pasting into terminal find doesn't work on Windows Jul 19, 2017
@Tyriar Tyriar added this to the Backlog milestone Jul 19, 2017
@levani
Copy link
Author

levani commented Jul 20, 2017

Also on macOS when I focus in the search input and click command+a the text in terminal is selected, not the input content. Pasting works fine though.

The problem seems to be that keyboard shortcuts don't always work in the context of the search input when it's focused...

@Tyriar
Copy link
Member

Tyriar commented Jul 20, 2017

The cmd+a thing is fixed in Insiders I believe.

@ihalip
Copy link
Contributor

ihalip commented Aug 24, 2017

This happens because TerminalPasteAction is assigned the Ctrl+V shortcut and it hijacks the input from the TerminalFindWidget.

@Tyriar, do you think changing the default keybinding to Ctrl+Shift+V would be ok? In this case, pressing Ctrl+V in the find widget works as expected.

@Tyriar
Copy link
Member

Tyriar commented Aug 30, 2017

It's ctrl+v as I've tried to stay in line with OS standards and that's what cmd.exe uses. This should be able to get fixed by resolving the conflict.

/cc @cleidigh @rebornix

@Tyriar Tyriar added the help wanted Issues identified as good community contribution opportunities label Aug 30, 2017
@bartsokol
Copy link

It also happens when you right click the search box, at least on Windows - see details described here.

@Tyriar Tyriar removed the windows VS Code on Windows issues label Sep 11, 2017
@Tyriar Tyriar modified the milestones: September 2017, Backlog Sep 11, 2017
@roblourens roblourens added the verified Verification succeeded label Sep 28, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 17, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug help wanted Issues identified as good community contribution opportunities terminal General terminal issues that don't fall under another label verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants