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

[WIP] change system clipboard library to a newer more maintained library #1487

Merged
merged 7 commits into from
Apr 19, 2017
Merged

[WIP] change system clipboard library to a newer more maintained library #1487

merged 7 commits into from
Apr 19, 2017

Conversation

xconverge
Copy link
Member

This library is being worked on consistently and is not stale with potential issues

Still having issues with the character • which I use for testing. Once it works perfectly with this I will also test chinese characters

@xconverge
Copy link
Member Author

Tested both this and current master (node-copy-paste) library on Ubuntu and all Chinese characters seem to work great. This new library is also cool because it worked out of the box, no need to install xclip. Back to macOS to continue to try and figure this out...

@xconverge
Copy link
Member Author

@rebornix this is starting to look great, but I wonder what should be fixed in the scope of this plugin, the copy paste plugin, or vscode

Since you are the copy paste formatting expert for vscode perhaps you can help me out here

Can you test this a bit? The key is mostly setting the LANG env variable for vscode to a UTF-8 compatible locale...

@rebornix
Copy link
Member

rebornix commented Apr 9, 2017

@xconverge sure, I'm a Chinese expert and I'll give it a try.

@xconverge
Copy link
Member Author

xconverge commented Apr 9, 2017

arg I can't get it to work for windows 10 :( , this test I just added fails on windows

@rebornix
Copy link
Member

@xconverge copying Chinese characters works good.

@xconverge
Copy link
Member Author

xconverge commented Apr 11, 2017

Too bad it doesn't work on windows yet :/ will see what I can do

@xconverge
Copy link
Member Author

I opened sindresorhus/clipboardy#11 we will see if it gets any traction

@xconverge
Copy link
Member Author

I am going to merge this for now actually since it FIXES the problem on osx and linux, uses a better supported library, and does not make windows worse (doesn't work correctly on master with old library either)

@xconverge xconverge merged commit 72b9c2b into VSCodeVim:master Apr 19, 2017
@xconverge xconverge deleted the change-system-clipboard-library branch April 19, 2017 19:29
@octref
Copy link
Contributor

octref commented Feb 19, 2018

@xconverge Hey wondering if Windows clipboard is still a standing issue? I want to push for microsoft/vscode#217.

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants