Mobile Integration - JCrop patch

My original plan was to pull out the code that made touch devices work with jcrop and integrate it with this modules cropping code. After looking at the Jcrop code it seemed like the touch code was in everything. I gave up on that and tried to get furf.com's touch-punch plugin to work. Which basically turned touch clicks into mouse clicks. I had it loading but it wasn't work.

Looking at his demo page I noticed he was running a different version of jquery UI. I found a patch for the jquery UI drupal module to run that version. Updated everything and it still didn't work.

So as a last resort I started converting this plugin to have an option to use Jcrop instead of the built in cropping. I Know zuuperman didn't want to totally change over to using Jcrop so inside the imagecache preset there is a checkbox that allows you to enable Jcrop for that preset.

I'm in the process of converting all the features over from the imagecache preset so everything will work correctly.

So far I've done:
crop width and height. working with #'s and percents. - I noticed on the original cropping the percent thing doesn't seem to work correctly.
offsets work with #'s or top,right,bottom,left,center. also made it so it uses that as a starting point now. instead of 0,0
resizable option is working
Do not allow down scaling is working correctly - modules original version seemed to not work correctly.
Still need to finish up aspect ratio.

also got it to correctly show previous crops. I will post a patch in another day or so.

Finally got a working patch. Every option inside the imagecache preset works with jcrop. Including aspect ratio & offset. In order to get the crop to use the xoffset and yoffset as a starting point I had to move some code up a little bit. It didn't seem to break anything. I tested the old crop as well to make sure nothing broke in the process of adding Jcrop option. I've tested on an Ipad & android phone and it does work. The only issue is you can not resize the page by pinching. That is an issue with how Jcrop detects touch events.

I tested this patch using the same jQuery Update 6.x-2.0-alpha1 module I needed for imagecrop. I did not need to upgrade juery. Also worth mentioning is Jcrop does not need jquery UI.

marinex is the issue you posted to related to this patch? Does this only happen after applying the patch?

Sorry about that patch taking out stuff that had nothing to do with the jcrop integration. I didn't notice it did that.
I did get a chance to test out the latest dev and it seems to be working good.
I am however getting an error when I run cron. But this is unrelated to the jcrop integration. I experienced this issue on the dev before you added jcrop.
When you run cron you get this error msg.

user warning: Unknown column 'ic.presetid' in 'field list' query: SELECT ic.fid, ic.presetid FROM imagecrop ic WHERE NOT EXISTS (SELECT fid FROM files f WHERE ic.fid = f.fid) AND ic.reference = 'files' in /Users/**/Sites/crop_nov_6/drupal-6.22/sites/all/modules/imagecrop/imagecrop.module on line 115.
user warning: Unknown column 'ic.presetid' in 'field list' query: SELECT ic.fid,ic.presetid FROM imagecrop ic WHERE NOT EXISTS (SELECT presetid FROM imagecache_action ia where ic.presetid = ia.presetid AND ia.action = 'imagecrop_javascript') AND ic.reference = 'files' in /Users/**/Sites/crop_nov_6/drupal-6.22/sites/all/modules/imagecrop/imagecrop.module on line 132.

I can open a new issue if you would like for tracking but I'm guessing its a quick fix.