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

Fix Windows clipboard behavior and improve documentation #89

Merged
merged 2 commits into from
Nov 4, 2022

Conversation

complexspaces
Copy link
Collaborator

This PR fixes the inconsistent behavior introduced by #60 on Windows that made arboard's semantics entirely different on Windows vs the other platforms, preventing reasonable cross-platform usage.

In addition, it improves the documentation of Clipboard to note that your app's usage of arboard may differ depending on what combination of platforms are being targeted. For example, if you're only targeting macOS and Windows, nothing beyond opening and using the clipboard in a one-off fashion is needed. However if Linux is used1, then one copy of the clipboard should be kept around at all times.

Closes #88

Footnotes

  1. The SetLinuxExt builder can help with some of these cases, but is not a universal solution for all apps.

Copy link
Contributor

@emilk emilk left a comment

Choose a reason for hiding this comment

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

Thanks for this 🙏 , and thanks for the new docs!

CHANGELOG.md Outdated Show resolved Hide resolved
This adds unit tests to back up the crate's documentation to
prevent silent regressions in the future.
Copy link
Collaborator

@ArturKovacs ArturKovacs left a comment

Choose a reason for hiding this comment

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

Thanks for this PR. I ran into this myself, I'm looking forward to seeing this merged.

@complexspaces complexspaces merged commit 189d675 into master Nov 4, 2022
@complexspaces complexspaces deleted the windows-clipboard-redo branch November 4, 2022 17:13
@complexspaces
Copy link
Collaborator Author

@ArturKovacs This is now released as 3.2.0 👍

@ArturKovacs
Copy link
Collaborator

Awesome! ♥

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.

Platform dependent use patterns of arboard
3 participants