Skip to content

Update OSC52 info for iTerm2 #3725

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ma-tecik
Copy link

As of iTerm 3.5.0, clipboard is fully supported. This PR updates the documentation.

Comment on lines +19 to +20
* `iTerm2`: supported — fully since 3.5.0, only copying (writing to
clipboard) in earlier versions.. Must be enabled in
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe improve the readability of your suggestion by:

Suggested change
* `iTerm2`: supported fully since 3.5.0, only copying (writing to
clipboard) in earlier versions.. Must be enabled in
* `iTerm2`: supported (fully since 3.5.0). In earlier versions copying (writing to
clipboard) must be enabled in:

@@ -16,9 +16,10 @@ Here is a list of terminal emulators and their status:
reading, add `read-primary` and `read-clipboard` to the
`clipboard_control` option.

* `iTerm2`: only copying (writing to clipboard) is supported. Must be enabled in
* `iTerm2`: supported — fully since 3.5.0, only copying (writing to
clipboard) in earlier versions.. Must be enabled in
`Preferences->General-> Selection->Applications in terminal may access clipboard`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

While we're at it we can remove the superfluous whitespace in a separate commit:

Suggested change
`Preferences->General-> Selection->Applications in terminal may access clipboard`.
`Preferences->General->Selection->Applications in terminal may access clipboard`.

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