RescueDog Posted August 25, 2013 Share Posted August 25, 2013 I've enable jqzoom in my shop and have discovered it is not functioning properly. 1.5.4.1 (upgraded from 1.5.0.1 via one-click) default theme knightsshop.com (tho I've disabled jqzoom now) Here's what I've discovered: Product with only ONE imageMouse over image results in broken graphic[*]Product with multiple images Mouse over default image results in broken graphic. Mouse over alt image, loads into primary frameMouse over alt image now in place results in zoomed image[*]Mouse over original (default) image to replace in primary box Mouse over orig in primary box now results in zoomed image. Conclusion...improper initialization? (Guessing, jqzoom is new to me) Again, I've turned jqzoom off. I'd like to investigate fixing it tho in hopes that others will find the information useful. ~Jo Link to comment Share on other sites More sharing options...
Jiten rash (away) Posted August 25, 2013 Share Posted August 25, 2013 can u show us ur site Link to comment Share on other sites More sharing options...
RescueDog Posted August 25, 2013 Author Share Posted August 25, 2013 knightsshop.com I'll enable for a couple of hours so you can see the issue. Link to comment Share on other sites More sharing options...
Jiten rash (away) Posted August 25, 2013 Share Posted August 25, 2013 (edited) omg did u modified projuct.js see wen u hover on a product for zoom they fetch a large image for zoom example large_default from images thats under backoffice preferences images make sure its enabled Edited August 25, 2013 by Jiten rash (see edit history) Link to comment Share on other sites More sharing options...
vekia Posted August 25, 2013 Share Posted August 25, 2013 something doesn't work well, when i hover mouse over the image, i see box with jqzoom, but it trying to load image: http://knightsshop.c...%20Sleeve%20Tee it is definitely wrong Link to comment Share on other sites More sharing options...
Jiten rash (away) Posted August 25, 2013 Share Posted August 25, 2013 see vekia..use dev tool and hover image to zoom in html filed u must see to src one is the image on product and the other is hidden one large image which zoom in this case i dont see src for zoom image Link to comment Share on other sites More sharing options...
RescueDog Posted August 25, 2013 Author Share Posted August 25, 2013 (edited) I haven't modified the code. The image must be there and enabled - because if you pick a product i.e. miscellaneous/mug and there are 3 pictures: black/red/blue - the default (black) will not allow zoom. If you change to red or blue - NOW the zoom works! Go back to black now and it too will work! I did a google search and came across other threads with this issue - eventually found this ==> https://github.com/PrestaShop/PrestaShop/commit/4e8c5a24dfa286614ed484756d19506ef0508f6e Am giving it a try now and will update post UPDATE: The solution above resolved the issue. One note - in my case the line #533 had to be completely added! ---THE ERRORED FILE WAS PREVIOUSLY UNMODIFIED FROM THE 1.5.4.1 package so others will experience this if the turn on jqzoom! Thanks for your help guys! Much appreciated!! Edited August 25, 2013 by topaward (see edit history) Link to comment Share on other sites More sharing options...
RescueDog Posted August 26, 2013 Author Share Posted August 26, 2013 Hi Guys - I discovered today that the fix I found only half helped. The zoom is now working but only with the default picture. So, if I have multiple views of the product, the left box changes on mouseover but it shows the default in the right. Hoping maybe someone more familiar can save me an hour of looking thru code. Thanks! Truly appreciate you guys! v 1.5.4.1 here's an example: http://knightsshop.com/prestakoc/clothing/10-button-up-oxford-dress-shirt.html Link to comment Share on other sites More sharing options...
RescueDog Posted August 26, 2013 Author Share Posted August 26, 2013 Final entry here - I pulled the file from github. zoom and image swap now working. Thanks guys! Link to comment Share on other sites More sharing options...
vekia Posted August 26, 2013 Share Posted August 26, 2013 thanks for infromation and your solution im going to mark this topic as [solved] best regards 1 Link to comment Share on other sites More sharing options...
Recommended Posts