MozillaFoundation / mofo-lightsaber

Giving every mofo a lightsaber in 2015
1 stars 0 forks source link

Add images #5

Closed OpenMatt closed 9 years ago

OpenMatt commented 9 years ago

plug-ins that could help:

ep_copy_paste_images Copy and Paste images from clipboard to a pad

ep_embedmedia Embed media (youtube, vimeo etc)
OpenMatt commented 9 years ago

This one is huge -- people would really love to be able to embed images. We got embedded videos working, but couldn't get images to work yet.

JohnMcLear commented 9 years ago

I did a LOT of work around images, a few weeks..

Images are a fsking nightmare in editors, especially those that are contentEditable.

I can implement basic image functionality but here are the things that users expect that are no longer being developed in contentEditable..

Trying to put a shim/solution in place that wraps handles around an image is a nightmare too as you end up with a lot of horrible cruft in the DOM.

There needs to be some consideration on if the image should be uploaded and stored on the file system too or if the base64 encoded content of the image should be saved as an changeset ergo in the HTML.. Saving Base64 would make export a lot easier (no external assets) but has it's own caveats...

Basically I can implement this as a plugin and maybe only allow image modification functionality in the browsers that support it.

It's at least a week dedicated work though..

jdotpz commented 9 years ago

ep_embedmedia is still behaving as we saw last time, with images being hosed but being able to embed other types of media.

JohnMcLear commented 9 years ago

I'm working on Image support now. See the copy paste images plugin. Support welcome ----- Reply message ----- From: "JP Schneider" notifications@github.com To: "MozillaFoundation/mofo-lightsaber" mofo-lightsaber@noreply.github.com Cc: "John McLear" John@mclear.co Subject: [mofo-lightsaber] Add images / videos / media to etherpads (#5) Date: Wed, Jan 28, 2015 16:24

ep_embedmedia is still behaving as we saw last time, with images being hosed but being able to embed other types of media.

— Reply to this email directly or view it on GitHubhttps://github.com/MozillaFoundation/mofo-lightsaber/issues/5#issuecomment-71864564.

jdotpz commented 9 years ago

screen shot 2015-01-28 at 10 31 29 am I got that as I tried to install the copy paste image plugin. Awesome that you are cooking on that!

JohnMcLear commented 9 years ago

You will need latest develop and it's a moving target. I only have one more days funding to finish the to-do so let's hope things go well later today .. ----- Reply message ----- From: "JP Schneider" notifications@github.com To: "MozillaFoundation/mofo-lightsaber" mofo-lightsaber@noreply.github.com Cc: "John McLear" John@mclear.co Subject: [mofo-lightsaber] Add images / videos / media to etherpads (#5) Date: Wed, Jan 28, 2015 16:31

[screen shot 2015-01-28 at 10 31 29 am]https://cloud.githubusercontent.com/assets/1373628/5941798/d9e15bec-a6d8-11e4-8f34-1f88cf25eea4.JPG I got that as I tried to install the copy paste image plugin. Awesome that you are cooking on that!

— Reply to this email directly or view it on GitHubhttps://github.com/MozillaFoundation/mofo-lightsaber/issues/5#issuecomment-71866186.

JohnMcLear commented 9 years ago

You can test image support here: https://beta.etherpad.org/p/imagedemo

jdotpz commented 9 years ago

Nice, it's looking great!

OpenMatt commented 9 years ago

@jdotpz should we mark this one closed?

OpenMatt commented 9 years ago
JohnMcLear commented 9 years ago

https://github.com/JohnMcLear/ep_copy_paste_images is the repo

Afaik I fixed copy/paste in FF

Please update Etherpad and the plugin, tnx :)

JohnMcLear commented 9 years ago

Yea it looks again like you are potentially running an old version, testing on https://beta.etherpad.org/p/image seems to show this issue resolved.

OpenMatt commented 9 years ago
OpenMatt commented 9 years ago

Noticing issues like this one: https://dl.dropboxusercontent.com/spa/6c38yp3crbxni5b/wk__vcww.png

JohnMcLear commented 9 years ago

@OpenMatt wow how did you make it do that? That looks like a browser bug, I'm running nightly here and don't see that.

OpenMatt commented 9 years ago

@JohnMcLear yeah that was probably a browser bug

jdotpz commented 9 years ago

+1