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

Add clipboard support for X11 samples #231

Merged
merged 2 commits into from
Aug 27, 2021
Merged

Conversation

barotto
Copy link
Contributor

@barotto barotto commented Aug 26, 2021

Now Linux users can play with the sandbox demo too!

@mikke89
Copy link
Owner

mikke89 commented Aug 26, 2021

Awesome! I had a look at this API at some point, and decided "nope". I'm glad you dived right into it! Kudos.

I had one issue while testing: It works well when I copy and paste between other windows, but trying to copy and paste within the RmlUi window itself does not work for me, it doesn't paste anything.

@barotto
Copy link
Contributor Author

barotto commented Aug 26, 2021

You're correct, I didn't check for that!
I'll fix it, should be easy enough.

@barotto
Copy link
Contributor Author

barotto commented Aug 27, 2021

I had a look at this API at some point, and decided "nope"

It's not that bad once you get the grasp of it. It's an event based system that allows the transfer of even gigs of data between apps without using system memory.
The problem is that documentation is mostly in the form of function reference...
In any case the Linux world is moving away from X11 and game developers should use abstraction libraries like SDL anyway.

@mikke89
Copy link
Owner

mikke89 commented Aug 27, 2021

Right, I do see how this API can be quite powerful. It's a lot more work for something basic, but I can appreciate the advantages too.

It works well for me now, thanks again!

@mikke89 mikke89 merged commit 9d071ce into mikke89:master Aug 27, 2021
@barotto barotto deleted the clipboard-x11 branch September 5, 2021 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants