Skip to content

Fix image (and other) pasting#411

Open
greg wants to merge 2 commits into
rsms:masterfrom
greg:fix-image-paste
Open

Fix image (and other) pasting#411
greg wants to merge 2 commits into
rsms:masterfrom
greg:fix-image-paste

Conversation

@greg

@greg greg commented Sep 23, 2017

Copy link
Copy Markdown
Contributor

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
Copy Markdown

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

@sebastienrospars

Copy link
Copy Markdown

I hope too @raxod502

@raxod502

Copy link
Copy Markdown

@rsms Ping?

@raxod502

Copy link
Copy Markdown

@rsms Is this project still maintained?

@greg

greg commented Aug 23, 2018

Copy link
Copy Markdown
Contributor Author

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

raxod502 commented Aug 23, 2018

Copy link
Copy Markdown

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

greg commented Aug 24, 2018

Copy link
Copy Markdown
Contributor Author

@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