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

Image pasting functionality into rooms #442

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

Conversation

b401
Copy link

@b401 b401 commented Oct 5, 2023

Hey there,

I've gone ahead and implemented image paste functionality into gomuks.
It seems like the better way to do this would get it done through mautrix instead but this is so far the working solution I've come up with.

I wasn't able to get Paste() or OnPasteEvent() to register my inputs so this is why I've added it to OnKeyEvent()

Thanks a lot for all your work on gomuks!

CTRL+V now checks if the clipboard contains an image and pastes it into the room.

The library used for clipboard functionality requires an additional
dependency:

> Linux: require X11 dev package. For instance, install libx11-dev or xorg-dev or libX11-devel to access X window system.
@b401 b401 changed the title add: clipboard functionality in room view Image pasting functionality into rooms Oct 5, 2023
@AlexBocken
Copy link

AlexBocken commented Dec 1, 2023

Thanks so much for developing this PR, I've been using your branch for the last few weeks without any issues.
Since using tulir's Whatsapp bridge ( https://github.com/mautrix/whatsapp ) I've come across an error which only persists in gomuks (Element for example does not have this error).

On pasting into Whatsapp-bridged chats I get the error:
image

Would you be able to insert a media type as well such that image pasting works for Whatsapp bridged chats as well?

Let me know if you need more information from me to resolvethis issue.

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