-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Implement virtual selection #207
Comments
Great. Adding this to the milestone for the next release. |
Removed this from the milestone as most important reason for this to be there was the right-click copy/paste regression, which is addressed now in #293 without virtual selection. |
I have a question: will it be possible then to have an automatic copy of selection into pasteboard? (as it don't seems to work at least on macOS) |
@shouze yep this can happen by utilising the hidden text area. if you are having trouble right now with copying and pasting in xterm.js can you open a new issue please to track what is going wrong there? |
@parisk I'll probably be able to pick this up in the next couple of months |
Here's what I believe needs to happen for this:
Unknowns
|
I don't think you'll be able to maintain perfect backwards theming functionality (you'll probably need to theme a css class, like CodeMirror does), but I don't think that should be a blocker. If necessary, you can do a major version point upgrade. It should only be a minute's work to migrate a style. |
Well the selection will likely be separate from the actual text, I'm thinking of just applying classes like the xterm-color-x ones to the selection as well. It would be a regression in VS Code is the main reason I'm after this, it's great for accessibility to invert the colors. I don't think it would be worth a major version update to change this, I'm the only one doing it AFAIK. |
@Tyriar since selection will have it's own DOM nodes, we can just use something like Last, there is no reason to issue a new major version for this, since we do not have an official, stable theming API yet. |
@parisk web selection |
This is part of the work to prepare for the upcoing selection changes Related xtermjs#207
When zoomed in line-height and actual rendered row height differs. This can be seen in the viewport being too large when zoomed in right now but also has additional impacts in the new selection model xtermjs#207
Handling selection manually, rather than relying on native web selection, will fix a bunch of issues that are introduced by the input helper text area, such as #68, #69 as well as:
.xterm
to the text area./
and similar chars as separators)The text was updated successfully, but these errors were encountered: