-
Notifications
You must be signed in to change notification settings - Fork 243
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 image (and other) pasting #411
base: master
Are you sure you want to change the base?
Conversation
- Screenshots and other copied image data now paste on the first try (at 99% JPEG quality, which is much smaller than PNG and will get ruined by Facebook anyway) - Copied file(s) and image(s) paste correctly - Errors when trying to paste a directory - Remove unnecessary MMFakeDragInfo dependency
@rsms any update on this? I'd love to see this long-standing bug fixed. |
I hope too @raxod502 |
@rsms Ping? |
@rsms Is this project still maintained? |
Doesn't look like it... personally, I've gone back to using messenger.com now that it supports image paste in Safari (one of the main reasons I had for using this app). The MIT license allows forking the project as long as the original author is left credited, so if someone feels like forking this and actively maintaining it, please go ahead and do that. (while you're at it, I also have #345 and #410 hanging for over a year, so you might want to merge those in too 😛) |
My use case was being able to jump unambiguously to Messenger with a global system hotkey. That would really be quite nice, but maybe I can learn to live without it. Alternatively, maybe https://messengerfordesktop.com/ works well enough to use instead. |
@raxod502 my issue with most messenger apps floating around (that one included, IIRC), was that they use Electron, which is a massive drain on RAM and battery, as opposed to this one which uses a plain WebView. |
The current paste implementation tries to override pasting image data with a fake drag event, which no longer works (requiring two attempts at a paste so it can paste the created URL normally the second time).
I've changed the logic so we mess around with the pasteboard data only when we have image data being pasted without a URL, and then everything gets passed straight through as a normal paste event with no further hackery.
MMFakeDragInfo
dependency