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

Fix image (and other) pasting #411

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

greg
Copy link
Contributor

@greg greg commented Sep 23, 2017

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.

  • 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), which resolves Need to paste images twice #393
  • Copied file(s) and image(s) paste correctly
  • Displays an error when trying to paste a directory
  • Remove unnecessary MMFakeDragInfo dependency

- 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
@raxod502
Copy link

@rsms any update on this? I'd love to see this long-standing bug fixed.

@sebastienrospars
Copy link

I hope too @raxod502

@raxod502
Copy link

@rsms Ping?

@raxod502
Copy link

@rsms Is this project still maintained?

@greg
Copy link
Contributor Author

greg commented Aug 23, 2018

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 😛)

@raxod502
Copy link

raxod502 commented Aug 23, 2018

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.

@greg
Copy link
Contributor Author

greg commented Aug 24, 2018

@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.

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.

Need to paste images twice
3 participants