I apologize before hand if this has either been requested before, or is in the development version as I don't have too much time to search through either of them to make sure.
I think that panels should include an extra "padding" classed div tag inside of the column to avoid breaking the layout. For example:
<div class="panel-col-first"><div class="padding">content</div></div>
That way panel-col-first / panel-col-last CAN have 50% width, and not break, considering the theme file takes advantage of the padding class (instead of the column class) in the css.
.panel-col-first { /* styles for column go here */ } .panel-col-first .padding { padding: 15px 20px; }
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | twocol_padding.zip | 1.4 KB | XBleed |
Comments
Comment #1
wim leersTo verify this, just create your own styles. (See the "styles" directory for examples.)
When you've got it working, submit it here so it can be considered for inclusion.
Comment #2
wim leersComment #3
XBleed commentedAlready ahead of you on that one. Here you are.
Note: I have only tested this in Firefox 2, IE7, and IE6 (but if it works in IE6, it'll HAVE to work in all others, right?) and it works perfectly.
It's only for two columns at the moment, as that is all I needed to create. I also took advantage of Drupals default "clear-block" class and got read of the
<br />clear reference.Comment #4
merlinofchaos commentedPanels 2.0 already does something like this, basically. I thought Panels 1.2 did too, but I don't remember how much of the CSS changed when I made the first conversion.
In any case, I am not considering patches against Panels 1.2 until a maintainer for the 1.x branch steps up.
Comment #5
wim leersI had not even noticed it wasn't for Panels 2.
Comment #6
sunSorry, Panels 1.x will not have any new major features. Consider upgrading to 2.x, please.