Skip to content

Adding X11 support#481

Open
victor-oliveira1 wants to merge 2 commits intod99kris:masterfrom
victor-oliveira1:master
Open

Adding X11 support#481
victor-oliveira1 wants to merge 2 commits intod99kris:masterfrom
victor-oliveira1:master

Conversation

@victor-oliveira1
Copy link

Hi there!

This is my first contribution on Github. Please forgive me if it's wrong.
I've added X11 support on clipboard pasting.

Thanks!

@d99kris
Copy link
Owner

d99kris commented Feb 11, 2026

Hi @victor-oliveira1 - thanks for contributing.
But I'm wondering what problem/bug the patch is fixing?

Clipboard copy/paste (incl. image) should already work on macOS and Linux X11 - enabled by library clip. It's only for Wayland that nchat has added special code (because clip does not support it).

I tested here now (Ubuntu on arm64), placing an image in the clipboard:

xclip -selection clipboard -t image/png -i doc/screenshot-nchat-dracula.png

and then pasting it in nchat (alt/opt-v) and the image can be sent.

@victor-oliveira1
Copy link
Author

Hi there!

I'm using X11 with openbox, and if i copy an image from web browser for example and paste into nchat with alt/opt-v, it simply does nothing.
With this fix, this works normally, i don't know why the current implementation doesn't works
I've tested on: Debian Testing (forky) + KDE, Debian Stable (trixie) + openbox
I'm going to debug and see why the clip lib it's not working here :)

Thanks!

@victor-oliveira1
Copy link
Author

Hi there!

I've made some tests and the clip lib (https://github.com/dacap/clip) it's not working here:
Captura_20260211

And with the build with the custom code provided:
Captura_20260211_000

@d99kris d99kris self-assigned this Feb 12, 2026
@d99kris
Copy link
Owner

d99kris commented Feb 12, 2026

Thanks - I'm travelling the next week, but I'll get back to this.

@fulalas
Copy link

fulalas commented Feb 13, 2026

I'm on Xfce 4.20 running X11 and no patch is needed to paste/send an image from the 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.

3 participants

Comments