Skip to content

Alt click should force regular selection in the Terminal #28863

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

Closed
joaomoreno opened this issue Jun 16, 2017 · 13 comments
Closed

Alt click should force regular selection in the Terminal #28863

joaomoreno opened this issue Jun 16, 2017 · 13 comments
Assignees
Labels
feature-request Request for new features or functionality help wanted Issues identified as good community contribution opportunities macos Issues with VS Code on MAC/OS X terminal General terminal issues that don't fall under another label verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@joaomoreno
Copy link
Member

I always use tmux with mouse support in Code. The mouse support prevents me from selecting text in the Terminal. I used to be able to hold Shift to bypass that behaviour and select text. That doesn't seem to work anymore. Did something change?

@Tyriar
Copy link
Member

Tyriar commented Jun 16, 2017

@joaomoreno literally everything changed 😉 xtermjs/xterm.js#670

Do you want to be able to do a regular selection when holding shift? Do other terminals let you do this? It looks like iTerm lets you use the regular (non-tmux) selection while holding alt/option, would this work for you?

@Tyriar Tyriar added feature-request Request for new features or functionality terminal General terminal issues that don't fall under another label labels Jun 16, 2017
@Tyriar Tyriar modified the milestones: Backlog, June 2017, July 2017 Jun 16, 2017
@jens1o
Copy link
Contributor

jens1o commented Jun 16, 2017

I'm not sure whether this addresses the same issue but it's hard to select a link which will open a file with a specific number(i.e. in error messages). You need to select at least one additional char after the link so you can copy it.

@Tyriar
Copy link
Member

Tyriar commented Jun 16, 2017

@jens1o this is one of the problems that is fixed with the new selection model 👍

@jens1o
Copy link
Contributor

jens1o commented Jun 16, 2017

Is this fixed in today's insiders? Yesterday, it wasn't fixed and I weren't able to check it out today.

@Tyriar
Copy link
Member

Tyriar commented Jun 16, 2017

@jens1o if you're seeing this in Insiders today please file an issue for more details 😃

@jens1o
Copy link
Contributor

jens1o commented Jun 16, 2017

No, it's fixed in today's insider. Sorry for the worries!

@joaomoreno
Copy link
Member Author

@Tyriar Wow I was unaware of the huge amount of changes. Great job!

Yeah I know iTerm used Alt for it and always found it weird that we use Shift, so Alt would be better!

@Tyriar Tyriar changed the title Shift select in Terminal doesn't work any more Alt click should force regular selection in the Terminal Jun 19, 2017
@Tyriar Tyriar added the macos Issues with VS Code on MAC/OS X label Jun 19, 2017
@Tyriar Tyriar added the help wanted Issues identified as good community contribution opportunities label Jul 7, 2017
@mjbvz mjbvz self-assigned this Jul 10, 2017
@Tyriar Tyriar modified the milestones: Backlog, July 2017 Jul 17, 2017
@joaomoreno
Copy link
Member Author

@Tyriar This is really painful 😢 I would love to attempt a PR. Gimme a hint. 👍

@Tyriar
Copy link
Member

Tyriar commented Jul 18, 2017

@joaomoreno it will involve a PR to xterm.js.

Currently we disable selection completely when entering mouse mode here. We need to keep it running but only respond when alt is down.

Current unknowns:

  • Does this work the same on Linux?
  • What happens if alt is dropped half way through the selection?
  • Does shift+alt click expand the selection?
  • Should it be an option?

@jens1o
Copy link
Contributor

jens1o commented Jul 29, 2017

@Tyriar Will this find its way into the July release?

@Tyriar Tyriar modified the milestones: July 2017, Backlog Jul 29, 2017
@Tyriar
Copy link
Member

Tyriar commented Jul 29, 2017

@jens1o yep, should do

@jens1o
Copy link
Contributor

jens1o commented Jul 29, 2017

YEAHHHHH

@Tyriar Tyriar added the verification-needed Verification of issue is requested label Jul 31, 2017
@Tyriar
Copy link
Member

Tyriar commented Jul 31, 2017

To verifier:

This feature is for macOS only, when in a tmux session with mouse mode on (selecting without a modifier should look different to the normal selection) holding alt should revert to the "regular" terminal selection, allowing the regular copy (cmd+c) to work.

@roblourens roblourens added the verified Verification succeeded label Aug 1, 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
feature-request Request for new features or functionality help wanted Issues identified as good community contribution opportunities macos Issues with VS Code on MAC/OS X 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

5 participants