• Resolved Kittar

    (@kittar)


    Hi,

    I’m not sure what you’ve changed in the recent versions of the plugin but it now breaks my layout. As far as I’ve seen, your plugin hooks to woocommerce’s action ‘woocommerce_before_single_product_summary’, which wasn’t the case before. My theme uses:

    remove_action(‘woocommerce_before_single_product_summary’, ‘woocommerce_show_product_images’, 20);

    and then calls:

    woocommerce_show_product_images();

    Later on in the template in a <div> tag.
    The expected behaviour should be that the default product images are simply replaced with SVI product images and it’s javascript. (as was the case with SVI a couple of versions ago)

    But now what happens is that SVI outputs the images on the above mentioned action, despite the fact I’ve removed ‘woocommerce_show_product_images’ from it. And what’s more bizzare it outputs an <h1> title upon calling woocommerce_show_product_images();

    Commenting out the said action:
    //do_action( ‘woocommerce_before_single_product_summary’ );

    it will output the images when manually calling woocommerce_show_product_images();
    BUT again, it outputs the product title for some reason as well.

    I hope I explained the issue clearly, please consider it.

    Thanks,
    Matt

Viewing 2 replies - 1 through 2 (of 2 total)
  • Thread Starter Kittar

    (@kittar)

    Followup: the title gets outputted because of my added action:
    add_action(‘woocommerce_before_single_product_summary’, ‘woocommerce_template_single_title’, 5);

    So ignore that part, the rest still stands – the images get outputted on:

    do_action( ‘woocommerce_before_single_product_summary’ );

    Despite:
    remove_action(‘woocommerce_before_single_product_summary’, ‘woocommerce_show_product_images’, 20);

    Plugin Author David Rosendo

    (@drosendo)

    Hi!

    I do this to ensure better theme compatibility.
    Some themes have their way to add this.

    Try disabling on your theme some “woocommerce_show_product_images” action

Viewing 2 replies - 1 through 2 (of 2 total)
  • The topic ‘SVI breaks layout’ is closed to new replies.