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

Pasting from image processor #2787

Closed
pjasiun opened this issue Apr 5, 2017 · 2 comments
Closed

Pasting from image processor #2787

pjasiun opened this issue Apr 5, 2017 · 2 comments
Labels
package:upload resolution:duplicate This issue is a duplicate of another issue and was merged into it. type:feature This issue reports a feature request (an idea for a new functionality or a missing option).

Comments

@pjasiun
Copy link

pjasiun commented Apr 5, 2017

It is possible to handle pasting from the image processor (like MS Paint), but it needs to be handled differently then pasting a file. In such case, data is not served as file but as <img> with data as base 64 string in the src attribute.

See the code which handles it in CKEditor 4: https://github.com/ckeditor/ckeditor-dev/blob/685fc7474fab779afe76edcf36011d2fb6bafab8/plugins/uploadimage/plugin.js#L68-L110

@Reinmar
Copy link
Member

Reinmar commented Jun 22, 2018

Let's close this as a DUP of https://github.com/ckeditor/ckeditor5-image/issues/211.

@Reinmar Reinmar closed this as completed Jun 22, 2018
@jiangyan33
Copy link

Upload a JPG image by pasting, request header content-type: image/ PNG.
Upload JPG images by clicking the upload button, request header content-type: image/jpeg.
I think the content-type value should always be image/jpeg.

@mlewand mlewand transferred this issue from ckeditor/ckeditor5-upload Oct 9, 2019
@mlewand mlewand added resolution:duplicate This issue is a duplicate of another issue and was merged into it. status:confirmed type:feature This issue reports a feature request (an idea for a new functionality or a missing option). package:upload labels Oct 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package:upload resolution:duplicate This issue is a duplicate of another issue and was merged into it. type:feature This issue reports a feature request (an idea for a new functionality or a missing option).
Projects
None yet
Development

No branches or pull requests

4 participants