It would be handy if an admin user could restrict the available presets per cck field. For example in the site that I am working on at the moment, when a user is adding/editing a store content type I know they will only ever want to use the store_full and store_thumb presets that I have created. Any other presets will not be used and are only going to confuse the user by being there.

Adding a "presets used" tab to the field admin page and then restricting the options when creating content would make this already great module even greater.

CommentFileSizeAuthor
#5 epsacrop_widget_settings.patch8.85 KBadr_p
#4 epsacrop_widget_settings.patch8.69 KBadr_p

Comments

jjwhitney’s picture

jjwhitney commented

I second this suggestion. This would be great. You could also remove the "[Manage image crops]" link if there are no "presets used".

Odd Hill’s picture

Odd Hill commented

This is an issue we are having as well. We have a site with around 20 different Imagecache presets, and now we can't see them in the popup window. It's also very difficult for end users to understand the presets since imagecache only uses machine-readable names without any descriptions.

We would love to see this implemented. We use EPSA crop on all our installations

adr_p’s picture

adr_p commented

Subscribing.

Is there already a way to narrow down available presets list? Like Odd Hill said, we also have a significant amount of imagecache presets and we simply cannot display them all on the form.

adr_p’s picture

adr_p commented
Status: Active » Needs review
StatusFileSize
new epsacrop_widget_settings.patch8.69 KB

As I couldn't wait for the response I prepared a patch for the module.

The patch adds a new setting to the imagefield widget. Now admin can narrow down presets that can be cropped for each field individually.

Patch also recovers _epsacrop_get_crop_actions(), that disappeard from the 6.x-1.0 version. I'm sure it was there a couple of days ago...

Please feel free to test the changes and send me your comments. I'm waiting also for the bug reports.

adr_p’s picture

adr_p commented
StatusFileSize
new epsacrop_widget_settings.patch8.85 KB

I've added an improvement and implemented a suggestion from jjwhitney to hide "[Manage image crops]" link where there are no presets enabled for a field.

You can get updated patch below.

yvmarques’s picture

yvmarques commented

Hello,

Patch commited :) Thanks !

Regards,

-- Yvan

andrew_k’s picture

andrew_k commented

Thanks guys, I'm looking forward to trying out this new feature

Odd Hill’s picture

Odd Hill commented

After trying out this latest addition the cropping has stopped working. I'm getting this from the console:

$.Jcrop is not a function
/modules/oddmodules/epsacrop/js/epsacrop.js?T
Line 53

bensey’s picture

bensey commented

I had the same problem.

Realised that when I replaced with the dev version I hadn't added the json2.js nor the jcrop libraries to the new folder. Also needed to clear the cache to get it all working.

Works well, great job (and a great module!).

jjwhitney’s picture

jjwhitney commented
Status: Needs review » Reviewed & tested by the community

This is working great for me. Thanks!

andrew_k’s picture

andrew_k commented

Its working great for me too. an awesome new feature!

yvmarques’s picture

yvmarques commented
Status: Reviewed & tested by the community » Fixed

I've just release new version with the fix.

Thanks eveybody

-- Yvan

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.