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

Document system clipboard option #272

Closed
wants to merge 1 commit into from
Closed

Document system clipboard option #272

wants to merge 1 commit into from

Conversation

karlhorky
Copy link
Contributor

@karlhorky karlhorky commented May 31, 2020

Document the system clipboard option introduced in #260.

Also, alphabetize the options for easier searching.


Set to `false` to keep VSCode's keybinding when searching.
Set to `true` to copy to and paste from the operating system clipboard.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a nitpick, but I'm not sure if "operating system clipboard" is always going to be correct. I'm thinking of cases like Linux where there isn't really an operating system clipboard. I think X11 still has a middle click clipboard and a Ctrl-C (system) clipboard. Using the term "system clipboard" would probably be more accurate here.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok sounds good, I had it as "system clipboard" before, and changed it to "operating system", thinking it would help less experienced folks. But let's use the simple one for now.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, i guess the question is who the reader is going to be. If they're installing a Vim extension, and not even the most popular one in the marketplace, they're likely to be a fair bit more techy than the average developer. I was pondering over in the other PR if perhaps we should call this "yank" for maximum vimishness. Not sure. Either way it's a nitpick, i guess the majority of readers will just be happy to see anything documented.

@karlhorky
Copy link
Contributor Author

Ah, already deleted the fork (too fast!) I've opened #274 for this.

@karlhorky karlhorky closed this May 31, 2020
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.

2 participants