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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Copying colored text while holding alt no longer works #2170

Closed
npezza93 opened this issue Jun 1, 2019 · 3 comments
Closed

Copying colored text while holding alt no longer works #2170

npezza93 opened this issue Jun 1, 2019 · 3 comments

Comments

@npezza93
Copy link
Contributor

npezza93 commented Jun 1, 2019

Hey there 馃憢!

Looks like something may have changed with selecting colored text in zsh from 3.13 to 3.14.
When copying colored text, pasting is throwing a bracketed-paste-magic error.

Details

  • Browser and browser version: chrome and safari
  • OS version: macos mojave
  • xterm.js version: 3.14.0

Steps to reproduce

  1. jump into zsh
  2. hold alt and select colored text
  3. copy highlighted text
  4. paste copied text

Videos showing the difference:

https://drive.google.com/file/d/1zMLVeNwrq7GaGuPj-FEMYzg60Oi_x6f_/view?usp=sharing
https://drive.google.com/file/d/1UXv6Hz-NhOmfPer7vhswA-7_DtVOrp0M/view?usp=sharing

Tyriar added a commit to Tyriar/xterm.js that referenced this issue Jun 1, 2019
@Tyriar
Copy link
Member

Tyriar commented Jun 1, 2019

I created an integration test for bracketed paste mode in #2172 and it seems to pass, not sure what's going wrong?

@npezza93
Copy link
Contributor Author

npezza93 commented Jun 4, 2019

Ok looks like my initial assumption was incorrect. This has nothing to do with copying colored text. I think it is due to 34ac1bc#diff-b28ef5ac37b020702c7a4f2e52676735. Currently in my app I automatically copy text when it is finished being selected. And from 3.13 to 3.14 it looks like the selection event gets fired when its cleared so basically a null string is being put on the clipboard which makes zsh unhappy. It totally makes sense to fire that event when the selection is cleared so this isnt an issue here. Sorry for the trouble!

@Tyriar
Copy link
Member

Tyriar commented Jun 5, 2019

@npezza93 thanks for letting me know, I didn't think that change would break anyone 馃檨

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants