Progenitor-Theme / progenitor

MIT License
7 stars 2 forks source link

Woocommerce #24

Open danielmrey opened 6 years ago

danielmrey commented 6 years ago

Hi Simon:

I am starting to add Woocommerce functionality, I took mainly all from your WBST.

When I used your WBST I took this JS code, maybe you can compare with the one that you have in functions/boostrap-4/bootstrap-4-wp.js and maybe add some of this code on board:


        // Adding Bootstrap classes to the Comments stuff
        $(".commentlist li").addClass("panel panel-default");
        $(".comment-reply-link").addClass("btn btn-default");

        // Forms
        $('select, input[type=text], input[type=email], input[type=password], textarea, input[type=tel], input[type=number]').addClass('form-control');
        $('input[type=submit]').addClass('btn btn-primary');

Thanks

danielmrey commented 6 years ago

Hi Simon:

This is in general the Woocommerce implementation:

2 Enqueues: wbst.css and wbst.js (DONE)

1 Function: woocommerce-setup.php I am calling this in functions/woocommerce/woocommerce.php (DONE)

1 Template: woocommerce.php I don't know where to put this file, it calls: <?php woocommerce_content(); ?> I think it needs to be in the root of the Theme, but I'd like to place it in woocommerce folder for encapsulation (PENDING)

Thanks

SimonPadbury commented 6 years ago

Ensure that woocommerce.php is up to date -- not from my old theme WBST unless it is still the same.

danielmrey commented 6 years ago

I think we do not need template as per the second way of adding Woocomemrce compatibility in this article (I think you used this in your WBST):

https://stackoverflow.com/questions/24283988/how-to-make-a-wordpress-theme-woocommerce-compatible?rq=1

danielmrey commented 6 years ago

Just committed all the files needed for Woocommerce, when you can can you give it a look?

Thanks.