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 copy on select option for the terminal #36271

Closed
ngryman opened this issue Oct 14, 2017 · 5 comments
Closed

Add a copy on select option for the terminal #36271

ngryman opened this issue Oct 14, 2017 · 5 comments
Assignees
Labels
feature-request Request for new features or functionality help wanted Issues identified as good community contribution opportunities terminal General terminal issues that don't fall under another label verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@ngryman
Copy link

ngryman commented Oct 14, 2017

Hi guys,

It's pretty much in the title, would you consider adding a copy on select option? If so I could open a PR.

Thanks!

@vscodebot vscodebot bot added the terminal General terminal issues that don't fall under another label label Oct 14, 2017
@Tyriar
Copy link
Member

Tyriar commented Oct 15, 2017

@ngryman are there other terminals that do this? Wouldn't the fact that it's attached to an editor make it much easier to mess up your clipboard accidentally?

@Tyriar Tyriar added the info-needed Issue requires more information from poster label Oct 15, 2017
@ngryman
Copy link
Author

ngryman commented Oct 15, 2017

are there other terminals that do this?

Yes, iTerm on OSX would be the most acknowledged.

Wouldn't the fact that it's attached to an editor make it much easier to mess up your clipboard accidentally?

Good point, but I don't think so for 2 reasons:

  1. It would be an opt-in, so if the user enables it, he is perfectly aware of the feature.
  2. It's copy on select, so it still involves a user action, that same user perfectly aware of the feature.

Those 2 reasons make a power user perfectly conscious of what is happening here.

The motivation for this feature is that when you actually select some text in your terminal, it's 99% of time to copy it and 1% because you have vision problems 🤓. Being able to copy on select simply removes an optional action for power users.

I would also suppose that VSCode users often copy/paste things from the integrated terminal to google errors or search a reference from an error stack in the code. So it makes perfect sense to offer them a way to simplify their life :)

@Tyriar Tyriar added help wanted Issues identified as good community contribution opportunities and removed info-needed Issue requires more information from poster labels Oct 18, 2017
@Tyriar Tyriar added this to the Backlog milestone Oct 18, 2017
@Tyriar Tyriar added the feature-request Request for new features or functionality label Oct 18, 2017
@Tyriar
Copy link
Member

Tyriar commented Oct 28, 2017

This is blocked on the upstream issue: xtermjs/xterm.js#1093

@Tyriar
Copy link
Member

Tyriar commented Nov 7, 2017

The upstream issue has been done and I just pulled in the change to vscode master, so this issue is unblocked and ready to be worked on.

Tyriar added a commit that referenced this issue Nov 7, 2017
Pulls in the selection event.

Part of #36271
@Tyriar Tyriar modified the milestones: Backlog, December 2017/January 2018 Dec 15, 2017
@Tyriar
Copy link
Member

Tyriar commented Dec 15, 2017

Added in #38287

@Tyriar Tyriar closed this as completed Dec 15, 2017
@Tyriar Tyriar added the verification-needed Verification of issue is requested label Jan 28, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Jan 31, 2018
@bpasero bpasero added the verified Verification succeeded label Feb 1, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality help wanted Issues identified as good community contribution opportunities terminal General terminal issues that don't fall under another label verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

3 participants