search

woocommerce / woosidebars

Replace registered sidebars in your WordPress themes using different conditions such as a specific page, a category or even an individual blog post.
https://wordpress.org/plugins/woosidebars/
51 stars 42 forks source link

Wrap widget in DIV by default #34

Closed leopuleo closed 9 years ago

leopuleo commented 9 years ago

Hi WooTeam, right now Woosidebars' sidebars, if they don't replace any original sidebars, show widgets wrapped in a list.

We've wrapped these in a <div>, for a better styling in theme. Please, consider to merge this feature in the master branch and WP repo.

Thanks!

mattyza commented 9 years ago

Hi @leopuleo.

Thanks so much for submitting this. :)

I left off these extra details on purpose, as these are inherited from the theme in use.

If you declare these details in your theme, when registering widget areas, you should be good to use a DIV for your sidebars without this addition. :)

Thanks and regards, Matty.

leopuleo commented 9 years ago

Hi @mattyza. no problen, I reported a bug to core, you can see how it's going on at https://core.trac.wordpress.org/ticket/30556 Cheers!