I've been experimenting with CloudZoom, and it's a great module! One issue I ran into is that it doesn't play so well with "touch devices".

I've found this: http://www.albanx.com/?pid=5&subid=17 which enables this. Would it be possible to add the patch to this module?

This version (http://pastie.org/private/04ok2bv5gexfpkvbrwvkq) fixes an iPad problem and works fine with the update here: http://drupal.org/node/1195864#comment-5782362

Comments

FranCarstens’s picture

Issue summary: View changes

Found improved, updated version of Javascript file.

satvision83’s picture

+1
It would be great to see this feature on the mobile devices.

FranCarstens’s picture

Satvision,

Using the information I found I did create a working version which can be found here:

http://drupal.org/node/1195864#comment-5992592

Not sure if there's been any progress outside of that.

FranCarstens’s picture

Issue summary: View changes

Forgot link

johnpitcairn’s picture

Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new17.96 KB

The problem here is that we need to patch an external library to make this happen with Cloud Zoom 1.0.3, which is no longer officially available. We can't submit changes upstream as a result, and there is probably no chance of getting this into the module. Setting to "needs review" anyway ;-)

That said, here is a patch incorporating the relevant changes from @FranCarstens (not the extra inline css changes) against libraries/cloud_zoom. It patches both cloud-zoom.1.0.3.js and cloud-zoom.1.0.3.min.js.

Tested so far with Commerce Kickstart 2 on Android 5 and iPad iOS7, seems to work.

To apply, navigate to the cloud zoom library folder and initialize a git repository:

cd [path/to/libraries]/cloud_zoom
git init
git add -A .
git commit -am 'before touch-enable patch'

Now download the patch file to the cloud_zoom directory and:

git apply -v

johnpitcairn’s picture

I have forked the cloud zoom 1.0.3 library with the touch support patch applied, get it from github:

https://github.com/johnpitcairn/cloud-zoom

dakku’s picture

Hi John. Thanks again for the patch. I am co-maintainer of this module and currently working on a drupal 8 port :)
I would like to include your patched version into the D8 module as an external JS dependency. Would you be ok with that?

johnpitcairn’s picture

Absolutely, go ahead.

jamesfk’s picture

Many thanks, John - your repo worked perfectly :)