Needs review
Project:
Imagefield Crop
Version:
7.x-2.x-dev
Component:
User interface
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
4 Aug 2011 at 11:56 UTC
Updated:
2 Oct 2011 at 16:36 UTC
Jump to comment: Most recent file
Comments
Comment #1
opdaviesHere's a much cleaner version...
Comment #2
yhager commentedWhy make it site-wide and not make it field specific?
Comment #3
opdaviesWhat do you suggest? Make it configurable for each specific image field?
Comment #4
yhager commentedIt makes more sense to provide this flexibility IMO.
Also note that 7.x-2.x is an experimental branch at the moment, and it is being maintained by zhgenti. It's better to base your work at this time on the 7.x-1.x branch.
Comment #5
iamjon commentedI created a patch against the 6.x branch of the module. If I am high-jacking the issue or should open a new issue please let me know.
To the best of knowledge the patch works but I think I'm missing something to make it work as it should.
Right now, I added another field to the settings form. I check if it has been clicked and surround the '// show dynamic crop previews' list function an if statement, avoiding the preview image output all together.
I wrapped the section in the js with a similar if statement.
But by avoiding the call to 'imagefield_crop_dynamic_preview' the drupal settings js looses all the predefined parameters of the orginal images.
To my surprise it worked although I need to refresh to actually see the changes.
Any ideas?
Thanks :)