Hi Rene,
Thanks for bringing this to our attention.
Yes, please do post this to the Roots support team.
From a WooSidebars standpoint, as long as the theme’s functionality somehow results in calls to dynamic_sidebar()
to display the widgets, and is_active_sidebar()
for checking if the sidebar has widgets, WooSidebars should handle just fine. 🙂
This information can be passed on to the Roots support team as well, please, if required. 🙂
Thanks and regards,
Matty.
Fixed with this commit last week: https://github.com/retlehs/roots/commit/714982fa42c09e63937a82f8c56819ae10a705c2
WooSidebars required do_action('get_header');
to work with Roots
Thanks retiehs.
As a side note, the “get_header” action is run when the “get_header()” function is called.
This is a standard WordPress template tag and is recommended as a WordPress theme standard. 🙂
Regards,
Matty.
We use get_template_part
to include those templates so that the theme can be better organized and grab files from a subdirectory. Roots isn’t a ‘standard’ WordPress theme:
https://github.com/retlehs/roots/blob/master/base.php
Nevertheless, while “get_header()” isn’t strictly required, as it doesn’t inhibit a theme from being a theme (and this is left to the discretion of the theme developer), it is important when making decisions of this nature to consider the implications there-of, such as removing a widely-used hook (ie: accounting for it with the appropriate do_action() call, as you’ve done in your recent commit).
Thanks for checking in and responding on this thread. I’m glad the issue has been resolved. 🙂
Hey guys, I have installed woosidebars, created widget area called “blog widgets… to replace the Primary ones. I add widgets to the new widget area and the widgets are not displayed…. Can you advise what might be wrong? I tried to uninstall the plugin and install again — did not help. I use canvas
I’d suggest you start your own thread on this forum – this one has nothing to do with Canvas theme and it’s 9 months old.