I'm looking for the best Media cropping solution out there, and right now this module is the most likely candidate. So, I'm going to do everything I can to help. I've gone through the (7.x) issue queue and answered any questions I could. I then played around with the module and filed any additional bugs/suggestions. Here's what the current path to a stable D7 release looks like. I'm hoping this will help the maintainer and those interested in contributing.

Bugs
#1416394: Issue when cropping new content
#1420494: Aspect ratio not being respected
#1392568: Add dimensions callback
#1471982: Class access error at imagecrop.admin.inc:209
#1272162: Page Not Found when cropping with Media Module
#1369290: "Crop this image" button does not appear when selecting image with Media selector
#1364636: Admin toolbar shows up in cropping colorbox

UI
#726666: Support for jquery_ui dialog
#1307204: Default crop
#1368402: Save Selection button not visible
#1491400: After uploading image with Media Selector, the cropping description gets in the way
#1491426: Crop button should not be wrapped in a <div>, should be places in a different order and should have it's text changed.
#1491494: Site theme used in popup instead of admin them

Documenation
#1387150: README file is missing

Features
#1250506: CDN support, new developer hook upon saving crop settings
#1113816: More flexible cropping mechanism?
#1361628: Add ability to use smallest image by default

Misc
#1421946: Combining Efforts with Duplicate Modules

Comments

mrfelton’s picture

mrfelton CreditAttribution: mrfelton commented

Have you checked the Manual Crop module? I used to use this module a lot, am a maintainer of it, but never use it any more. Manual Crop does a much better job, IMO, and it has Media module integration on the way. Worth checking out if you haven't already. I for one would rather see effort go into that module.

mrfelton’s picture

mrfelton CreditAttribution: mrfelton commented
Issue summary: View changes

Cleanup

bryancasler’s picture

bryancasler CreditAttribution: bryancasler commented

Thanks mrfelton, I'm open to all options. I'll give it a test drive right now.

mrfelton’s picture

mrfelton CreditAttribution: mrfelton commented

@animelion - you may have slightly misread my comment... I'm a maintainer of Image Javascrip Crop. A very very inactive maintainer! Why? Because I think that Manual Crop beats this solution hands down, it is slick, well maintained, and is heading in the right direction. I pretty much abandoned Image Javascrip Crop as soon as I found the Manual Crop module. Install it, and you'll see what I mean ;)

bryancasler’s picture

bryancasler CreditAttribution: bryancasler commented

Looks great! I'll move my efforts over there. The big problem right now is that it's not saving the crop. I'll follow up over there. Thanks a bunch!

bryancasler’s picture

bryancasler CreditAttribution: bryancasler commented
Status: Active » Closed (won't fix)

Thanks again mrfelton I'm closing this issue.

Heading over here!
#1491604: Support for the "Media file selector" widget

nils.destoop’s picture

nils.destoop CreditAttribution: nils.destoop commented
Status: Closed (won't fix) » Active

For the record. This module is still maintained ;)
All media errors should be fixed by last commit.

mrfelton’s picture

mrfelton CreditAttribution: mrfelton commented

I know it's still maintained, and there has been a lot of good work thats gone into this project - but... I think it's worth considering wether there needs to be several different solutions to the same problem, or wether one solution and one united effort would yield a better end result.

bryancasler’s picture

bryancasler CreditAttribution: bryancasler commented
Title: A call for consolidating efforts. The path to a media integrated D7 cropping module. » The path to a stable D7 release.

There are four D7 cropping modules that currently integrate with the media module. And they are all being actively maintained. I want to note that Media crop is unique in that it works only through WYSIWYG and no other module offers that right now. As WYSIWYG support is integrated into each cropping module, each module will use its own markup, re-enforcing why we should combine efforts now before potential fragmentation occurs.

Media Integrated Cropping Modules

  1. Manual Crop
  2. Image javascript crop
  3. EPSA Crop - Image Cropping
  4. Media crop

Manual Crop

D7 Users: 418
Last D7 commit: 3/19

Image javascript crop

D7 Users: 1632
Last D7 commit: 3/21

EPSA Crop - Image Cropping

D7 Users: 684
Last D7 commit: 3/21

Media crop

D7 Users: 110
Last D7 commit: 3/19

Cropping modules that do not yet integrate with the Media module

  1. Imagefield Crop
  2. ImageField Focus
  3. Media Image Transform

Assuming "Image javascript crop" has been mostly fixed in its latest commit, still need to test that, then all these modules work to some varying degree. I just got finished testing "EPSA Crop" and it's working 100%. I don't see that any of these modules are "feature complete", but that's another discussion to have after this one.

As mrfelton said, should we combine efforts? or would we be better off developing separately? I believe that there is at least some efforts that could be combined. I'm not a maintainer, but as a user I do have invested interest in seeing wide adoption of any module I use. I'm going to post a link to this conversation in the other module's issue que's. Hopefully this will get all parties talking with each other.

Additional Reference:
Comparison of image cropping and resizing modules
#1291518: Use Field Formatters for Embedding Options
#1451316: Clean up wysiwyg-media.js
#1283844: [meta] Improve WYSIWYG integration

bryancasler’s picture

bryancasler CreditAttribution: bryancasler commented
Title: The path to a stable D7 release. » A call for consolidating efforts. The path to a media integrated D7 cropping module.
grendzy’s picture

grendzy CreditAttribution: grendzy commented

Can you clarify what you mean by "Non-media Integrated"? In my testing Smart Crop is compatible with media, as are all image style effects.

bryancasler’s picture

bryancasler CreditAttribution: bryancasler commented

My apologies grendzy, smart crop shouldn't be included in this list. I've made a correction to my post.

vood002’s picture

vood002 CreditAttribution: vood002 commented
Title: The path to a stable D7 release. » A call for consolidating efforts. The path to a media integrated D7 cropping module.

Just my 2cents: I just tried to get Image Javascript Crop to work and after about an hour had no success. I switched over to Manual Crop, it worked out of the box, and I like it's approach to Media integration more (a work in progress) in that it offers the cropping selection on file entity edit pages.

gooddesignusa’s picture

gooddesignusa CreditAttribution: gooddesignusa commented

Don't forget ever since we switched 'Image javascript crop' over to use JCrop we can now crop on mobile devices.

bryancasler’s picture

bryancasler CreditAttribution: bryancasler commented

To build on what gooddesignusa said. The "Imagefield Crop" module is already using the jcrop library, but as mentioned in #8 it doesn't have media support just yet.

nils.destoop’s picture

nils.destoop CreditAttribution: nils.destoop commented

@vood002 And can you explain what you didn't get to work? I can set it up in less then 5 minutes.
The cropping selection for media is done on both file entity edit, as the file selector.

vood002’s picture

vood002 CreditAttribution: vood002 commented

@vood002 And can you explain what you didn't get to work? I can set it up in less then 5 minutes.
The cropping selection for media is done on both file entity edit, as the file selector.

Certainly. I'm running it on one other site...a beta version I remember, so I have had success with this module before...but in this case it appeared to be having some javascript errors. The option to 'scale image to fit window' wasn't working, I found an issue about this in the queue and applied the patch I found but this still didn't solve this issue for me.

Even beyond this initial scaling error the jcrop functionality simply wasn't working...I would provide a screenshot but I've already uninstalled. On initially opening the image for cropping I could see one handle in the upper right left and it looked like the cropping field was at least twice the size of the image I was looking at since I couldn't see any other handles. I was unable to click and drag this handle, or to resize the box myself. I double checked the settings in Styles and they all seemed to be set properly. I assumed a compatibility error with some other script on my site, though it wasn't throwing any errors. I didn't investigate too far since I had success with Manual Crop.

FWIW I'm certainly not saying this is a bad module, and I could see the extra configuration options that ijc provides being useful in certain circumstances...but I just wanted to chime in with my experience since I just tested both modules and this is a pertinent discussion.

vood002’s picture

vood002 CreditAttribution: vood002 commented
Issue summary: View changes

Adding a bug

nils.destoop’s picture

nils.destoop CreditAttribution: nils.destoop commented
Status: Active » Closed (won't fix)
nils.destoop’s picture

nils.destoop CreditAttribution: nils.destoop commented
Issue summary: View changes

type

slashrsm’s picture

slashrsm CreditAttribution: slashrsm commented
Issue summary: View changes

FYI: #2368337: Drupal 8 port (duplicate)