middlesister/thematic-html5 — pull request #7 by scottnix (2012-08-24 03:58:40+00:00)
Title: Add a div and class to widget output.

While converting one of my child themes to be compatible with your plugin, I noticed one thing that isn't required, but would be helpful.

In the current version, the ul is removed that wraps the widget areas, which previously madeaa perfect wrapper for inside the sidebar. Currently the way the box model is, it is always easier to put padding on internal elements, but there is no internal element that spans the sections inserted to widgets.

Just a thought, while it would be helpful to me (and I would think plenty of others), I kind of just wanted to play with Github and do my first pull request.

Feel free to deny me, but think about it, the current box model sucks, if you have a sidebar that is 300px wide which fits perfectly, and then you add a 2px border, it will be 302 wide and drop below. Now if there was just a internal element inside it, you could use that instead. :)