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

Support TinyMCE #52

Open
Misiu opened this issue Jan 28, 2015 · 8 comments
Open

Support TinyMCE #52

Misiu opened this issue Jan 28, 2015 · 8 comments

Comments

@Misiu
Copy link

Misiu commented Jan 28, 2015

First I want to say this is an awesome plugin, I hadn't test it yet, but from demo I can see this is THE functionality I was looking for.
Thank You for sharing this awesome piece of code.
I noticed that You have CodeMirror integration, do You plan to add tinyMCE support? right now when placing images into TinyMCE they are included as base64, instead I would like to include them like You plugin is doing.

Maybe You already used Your plugin with TinyMCE?

@Rovak
Copy link
Owner

Rovak commented Jan 28, 2015

Good one, gonna look into it this weekend!

@Misiu
Copy link
Author

Misiu commented Jan 28, 2015

@Rovak awesome news :)
Hope this won't be to complex to implement.

@Rovak
Copy link
Owner

Rovak commented Feb 7, 2015

I've been doing some tests and implementing simple behavior is easy. I think the exceptions are gonna be harder to implement in the editor because the pasted data can contain more complex situations:

  1. Content with multiple images
  2. Content with images which are already uploaded, they should not be uploaded again.
  3. Broken Image links
  4. External image links

I will try to implement the simple behavior first. After that i will consider implementing the more complex situations.

@Misiu
Copy link
Author

Misiu commented Feb 11, 2015

@Rovak, awesome news :)

  1. Multiple images should be processed one by one, I tried adding text (with images) from Word document into GitHub and images are not pasted, just text
  2. I think this might be addressed by server side. When uploading PHP could count checksum from image and store filename and checksum on server (simple database) then when someone will try to upload same image (even with different filename) we could check if we have checksum in database and return image link for that checksum. Maybe a simple idea, but this could save space on server for duplicates.
    3 & 4 I don't know how TinyMCE is doing that, but if we paste link to external image it should stay as is, or maybe we could try to fetch that on server side and return link to our local image? In case of any errors we could return default error image.

What do You think?

@Misiu
Copy link
Author

Misiu commented Apr 20, 2015

@Rovak any news on this?

@Rovak
Copy link
Owner

Rovak commented Apr 24, 2015

not yet, been busy with other things and this one isn't very high on my priority list right now

@Misiu
Copy link
Author

Misiu commented Aug 13, 2015

Hi Rovak, any chances You might look into this issue?

@Rovak
Copy link
Owner

Rovak commented Aug 13, 2015

Hi Misiu, like i mentioned before, this issue is not very high on my priority list, this plugin is focused on inputs which are not WYSIWYG based like textarea's and markdown syntax. TinyMCE already has a way to embed images so i doubt that this plugin will be used much in combination with TinyMCE.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants