Hi, I've notices the request for a feature to edit the crop one just did because it was wrong.
I'm using media_crop together with media-7.x-2.0 and ckeditor-4.0.

I see some problems that needs to be solved for this to happen:

1. When editing the image within the wysiwyg it is impossible to know from the file which MCIID it had, so one cannot get the just made configuration.
2. There is some code for the ckeditor plugin, that what I can see doesn't do anything.
3. Old crops need to be removed in the filesystem and updated in the db, right?

Another issue is the interface for the cropping is not perfect, I'd love to help improve it if these issues above can be solved.

Best regards, Tim

CommentFileSizeAuthor
#12 2130785-media-crop-redesigned.patch92.55 KBtimbrandin
#7 media-crop-2.x.png352.78 KBtimbrandin
#4 Screen Shot 2013-11-11 at 09.51.00.png352.78 KBtimbrandin
#3 Screen Shot 2013-11-09 at 15.33.18.png363.8 KBtimbrandin

Comments

timbrandin’s picture

timbrandin commented
Title: Editing cropping already done. » Editing of a crop
timbrandin’s picture

timbrandin commented
Issue summary: View changes
timbrandin’s picture

timbrandin commented
StatusFileSize
new Screen Shot 2013-11-09 at 15.33.18.png363.8 KB

I've added an image on how I've reworked it locally. How about it?

timbrandin’s picture

timbrandin commented
Title: Editing of a crop » Version 2.x - Scale/Crop redesigned.
StatusFileSize
new Screen Shot 2013-11-11 at 09.51.00.png352.78 KB

So how about this.
Only local images are allowed.

timbrandin’s picture

timbrandin commented

Anyone still interested in this? I have a patch almost ready that I can post here.
But soon if no one is answering I will post this as a new project.

Best regards,
// Tim, Findwise

timbrandin’s picture

timbrandin commented
Priority: Normal » Major
timbrandin’s picture

timbrandin commented
StatusFileSize
new media-crop-2.x.png352.78 KB

Media Crop 2.x

bgilhome’s picture

bgilhome commented

That looks great, much desired functionality and compatibility! What are your plans re: patch vs project?

parrapa’s picture

parrapa commented

Great work! It looks exactly what I was looking for. Do you have an estimation of release?

Thanks!

rodpal’s picture

rodpal commented

Absolutely!

timbrandin’s picture

timbrandin commented

We have it in production on two clients now so its just me being lacy making a patch, will do so ASAP.
// T

timbrandin’s picture

timbrandin commented
StatusFileSize
new 2130785-media-crop-redesigned.patch92.55 KB

Added a patch that does the work, might be some bugs in there but who is perfect.

thamas’s picture

thamas
Primary language Hungarian
Location Hungary
commented

Tested with
Media 7.x-2.0-alpha3
Media Crop 7.x-1.x-dev
WYSIWYG 7.x-2.2

It not works correctly for me. Image editor modal is not wide enough for bigger images and there are no scrollers to. Image styles not applied. Images added to content using earlier (2.x) version of media and Media Crop 1.4 are not displayed.

kenorb’s picture

kenorb commented
Status: Active » Needs work