-
-
Notifications
You must be signed in to change notification settings - Fork 881
desktop: Make clipboard support optional #15535
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
Conversation
ea2dabb
to
b7a8592
Compare
On wayland ruffle only supports `wlr-data-control-unstable-v1` which is not intended to be supported by all compositors.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
We really should get to the bottom of this Wayland protocol mess someday...
Second! I really hope |
Well, of course... 😅 |
Oh, and welcome, new contributor! 👋 😄 |
Speaking of - could you please raise this to https://github.com/Smithay/smithay-clipboard/issues as well, @ColinKinloch? |
Smithays fine, I'm currently looking into whether I can use it in place of arboard. On that, do you have any example swf files that access the clipboard? |
I meant the |
Yeah, see: #6476 (comment) |
In wayland the clipboard is defined in the core protocol by
"primary selection" refers to the ability to use middle click to paste the last highlighted text.
|
On Wayland ruffle uses https://github.com/YaLTeR/wl-clipboard-rs which only supports
wlr-data-control-unstable-v1
which is not intended to be supported by all compositors.There are options with better support https://wayland.app/protocols/primary-selection-unstable-v1 but until it's implemented in
wl-clipboard-rs
ruffle will crash for want of clipboard access.For most platforms this MR wont effect anything. On platforms without clipboard support it'll quietly fail to set and get the clipboard.
Fixes #15213
Mutter doesn't intend to support it: https://gitlab.gnome.org/GNOME/mutter/-/issues/524