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

Copy from Terminal Selecting Wrong Value #2850

Closed
cancan101 opened this issue Sep 18, 2017 · 7 comments · Fixed by #3189
Closed

Copy from Terminal Selecting Wrong Value #2850

cancan101 opened this issue Sep 18, 2017 · 7 comments · Fixed by #3189

Comments

@cancan101
Copy link
Contributor

I have the following on my terminal:

jjia@node002-jupyter-20170918-141251:/usr/src/app$ date
Mon Sep 18 15:27:19 EDT 2017
jjia@node002-jupyter-20170918-141251:/usr/src/app$

but when I try to copy and then paste the first two, I end up getting:

Mon Sep 18 15:27:19 EDT 2017
jjia@node002-jupyter-20170918-141251
@cailiang9
Copy link

cailiang9 commented Sep 19, 2017

xtermjs/xterm.js#910 is waiting to be merged with xterm.js .

@parente
Copy link
Member

parente commented Sep 19, 2017

Quick experimentation for a hacky workaround while things get fixed upstream. Replacing this rule:

https://github.com/jupyter/notebook/blob/master/notebook/static/terminal/less/terminal.less#L24-26

with:

.xterm-rows {
  top: 10px;
  left: 10px;
}

and adding rule:

.xterm-selection {
  top: 10px;
  left: 10px;
}

seems to work around the problem with xterm 2.8.1 included in the notebook 5.1 release.

@cancan101
Copy link
Contributor Author

@parente do you want to pr that change?

@takluyver
Copy link
Member

Is working around it going to cause any problems when it's fixed in xterm? I don't think we have a great way to ensure that the workaround is removed when we upgrade it.

@oeway
Copy link

oeway commented Nov 12, 2017

@cancan101 Could you please fix this issue? xtermjs/xterm.js#910 seems ready. It's a bit annoying for copy and paste from the current terminal. Thank you.

@cancan101
Copy link
Contributor Author

V3 hasn't been released yet. Should be any day

@cancan101
Copy link
Contributor Author

cancan101 commented Jan 5, 2018

New version of xterm (v3.0.1) has been released. That being said, they have dropped support for bower. Any chance to pull in dependencies from npm / yarn?

Looks like until this PR, xterm was loaded from npm.

cancan101 added a commit to cancan101/notebook that referenced this issue Jan 8, 2018
@minrk minrk added this to the Reference milestone Sep 13, 2018
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 7, 2021
echarles pushed a commit to datalayer-externals/jupyter-notebook-classic that referenced this issue Apr 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants