Woocommerce Composite Images - by RR123 - on WordPress WooCommerce support

This topic has 19 replies, 3 voices, and was last updated 7 years, 4 months ago ago by Olga Barlow

  • Avatar: RR123
    RR123
    Participant
    November 25, 2016 at 14:29

    Hi there,

    We just noticed that the lightbox does not work on images displayed for chosen selected components for a composite product. If I take the very same product and view it again after switching to Storefront, all is well and the lightbox opens.

    That’s a big problem because these images are really tiny and then destroy the usage flow by opening in the same window, thereby losing settings/position, etc. in the user’s composite construction

    To help, you should know that the opening range around the culprit image has the following construction:

    The data-rel attribute seems to show in Xstore, so I guess you need to know the class (composited_product_image zoom) to make it work?

    Thanks again,

    Regards

    Please, contact administrator
    for this information.
    18 Answers
    Avatar: RR123
    RR123
    Participant
    November 25, 2016 at 16:01

    Hi there,

    The same seems to be true of WooC Bundles. Same problem of failing lightbox.

    Their class is:

    bundled_product_image zoom

    and full tag:

    <a href="http://blahblah.jpg" class="bundled_product_image zoom" title="blah" data-rel="prettyPhoto">

    Avatar: RR123
    RR123
    Participant
    November 26, 2016 at 18:26

    Actually, this is very critical, more than we realised.

    It completely breaks the JS when the users goes back to the composite builder. They have to totally refresh and start again……

    Avatar: 8theme
    8theme
    Support staff
    November 28, 2016 at 08:53

    Hi,

    Could you please check the admin access? We can’t login your website and see the problem at the moment.

    Regards

    Avatar: RR123
    RR123
    Participant
    November 28, 2016 at 11:05

    Sorry, I didn’t share it. Bear with me. I am just getting you the details…

    Avatar: RR123
    RR123
    Participant
    November 28, 2016 at 11:12

    Details in the private info. Please can you let me know what you do to sort it. The details are for a dev staging area and I will need to implement on the production.

    Thank you.

    Please, contact administrator
    for this information.
    Avatar: RR123
    RR123
    Participant
    November 28, 2016 at 11:14

    Direct URLs that show problem on the way too…. 2secs….

    Avatar: RR123
    RR123
    Participant
    November 28, 2016 at 11:24

    Examples in private details

    Please, contact administrator
    for this information.
    Avatar: 8theme
    8theme
    Support staff
    November 28, 2016 at 14:51

    Thank you.

    Firstly, you must fix the following JS errors that occur in your JS console and may break some functionality.
    https://gyazo.com/c3447a35226a171e329285fe97fd914c

    Regards

    Avatar: RR123
    RR123
    Participant
    November 28, 2016 at 15:34

    Hello,

    Those console messages are not the issue, for a few of reasons I can be sure of.

    The error path shows that these are messages from one of the hosting company’s ‘must use’ adminbar plugins. They are not of particular consequence, but more importantly this means that there are no console errors of any kind when not a user is not logged into an admin account or any account. There are no JS errors at all. But the light box problems still exists for composites and bundles.

    The other point is that lightbox is working properly for the main product image you see at the top left (even with the console messages you highlighted). Just disable the ‘cursor’ and ‘pointer event’ CSS in firebug/dev tools for that image and you will see the lightbox work nicely.

    I hope that helps.

    R.

    Avatar: 8theme
    8theme
    Support staff
    November 28, 2016 at 16:01

    Ok, we just checked CSS classes for this image and see that composite_product_image zoom is there (you mentioned it in the first message) http://prntscr.com/dcu6k9
    It means that our theme doesn’t change any markup for this plugin. Does it have any additional requirements that may be missed in Xstore?

    Regards

    Avatar: RR123
    RR123
    Participant
    November 28, 2016 at 17:15

    Hi there,

    I know that class was there – that’s why I was highlighting it to you. That way you could find the correct element efficiently and see that the lightbox does not work despite the correct markup i.e. the class and pretty attribute being there correctly.

    In short and to recap, it works when you move to storefront………….

    This is feeling a little circular today………

    Avatar: RR123
    RR123
    Participant
    November 28, 2016 at 18:07

    Hello,

    I have just added sftp details to the private content

    Please, contact administrator
    for this information.
    Avatar: 8theme
    8theme
    Support staff
    November 29, 2016 at 08:49

    Hello,

    Ok, please check now. We have returned back prettyphoto scripts from woocommerce. All changes made in child theme functions.php file.

    Regards

    Avatar: RR123
    RR123
    Participant
    November 30, 2016 at 12:23

    Wonderful Serg – thank you. I didn’t have a chance to reply yesterday.

    Best wishes,

    R

    Avatar: RR123
    RR123
    Participant
    November 30, 2016 at 12:31

    I forgot to ask, will this fix be in future updates?

    I don’t want duplicate cpu clock cycle usage when it does.

    thanks again

    Avatar: Olga Barlow
    Olga Barlow
    Support staff
    November 30, 2016 at 15:22

    Hello,

    We can’t include this fix for the main theme because we don’t use prettyphoto scripts for the image gallery by default, we use our own lightbox.

    All the fixed to change it back were made in functions.php of your child theme, so you won’t lose it after update if you don’t change our code in functions.php.

    Regards

    Avatar: RR123
    RR123
    Participant
    November 30, 2016 at 16:40

    Hi there,

    Understood, but confused… I didn’t realise you use a different lightbox.

    What is the difference?

    Avatar: Olga Barlow
    Olga Barlow
    Support staff
    December 1, 2016 at 16:58

    Hello,

    You may check our demo and compare how looks popup on your site and on our demo http://prntscr.com/de3kq7

    Regards

  • Viewing 19 results - 1 through 19 (of 19 total)

You must be logged in to reply to this topic.Log in/Sign up

We're using our own and third-party cookies to improve your experience and our website. Keep on browsing to accept our cookie policy.