Page 1 of 1

[2.0] Image editor

Posted: Fri Jun 28, 2013 3:00 pm
by yellowled
So there's one thing in the 2.0 backend which I'd like to discuss: the backend includes a rather rudimentary image editor, basically for stuff which GDlib or imagemagick can do server-side. You know, cropping, resizing, rotating. Actually, resize and rotate are not part of the actual image editor but seperate actions in the media db.

I understand the actual image editor is kind of “proof of concept” and/or unfinished work. I'd vote to remove the image editor altogether, at least for the time being. That might, however, be influenced by the fact that I would never do that kind of stuff (especially rotate and crop) in the s9y backend. I would always use a proper image editor like GIMP or Photoshop for that.

My main concern here is that we would ship unfinished parts with 2.0 unless someone were to improve the image editor significantly which, as far as I can tell, would be quite an effort. I also think that it's “hidden well enough” (it's only available in the media properties page) to remove it (temporarily).

Agreed, opposed, comments?

YL

Re: [2.0] Image editor

Posted: Fri Jun 28, 2013 3:25 pm
by onli
Agreed. I didn't even know that thing exists till i stumbled upon it while coding.

Re: [2.0] Image editor

Posted: Fri Jun 28, 2013 5:11 pm
by bernd_d
Sometimes i use "Rotate" because of the reason, that the current library doesn't use the exif-orientation-tag. My locale image-viewer auto-rotate pictures for viewing and after uploading them, they are wrong/not rotated.

Cropping and other things could be removed.

Re: [2.0] Image editor

Posted: Fri Jun 28, 2013 5:19 pm
by onli
Sometimes i use "Rotate" because of the reason, that the current library doesn't use the exif-orientation-tag
Should we patch this in 2.0?

Re: [2.0] Image editor

Posted: Fri Jun 28, 2013 6:01 pm
by bernd_d
Don't know. If so, it should be optional like a check-button for upload-manager. Manual rotating should still be implemented, but one single button for an image could be enough (for example "clockwize"), a true image-editor is not necessary.

Re: [2.0] Image editor

Posted: Mon Jul 01, 2013 1:27 pm
by garvinhicking
Hi!

I developed that a long time ago, I just wanted to put it somewhere. I have nothing attached to it. Feel free to remove it. I don't think it works anyways, and with the advent of jquery, the whole thing would be done differently.

Regards,
Garvin